Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow selective disclosure with ops other than $eq. #65

Open
nedgar opened this issue May 1, 2023 · 12 comments
Open

Allow selective disclosure with ops other than $eq. #65

nedgar opened this issue May 1, 2023 · 12 comments
Labels
enhancement New feature or request

Comments

@nedgar
Copy link

nedgar commented May 1, 2023

The current support for selective disclosure is indicated by query {fieldName: {}}, which implicitly assumes the operator is $eq, with value being the current value in credential.

If selective disclosure was purely an option, and the other operators were allowed, it would support a challenge like “prove you’re an adult and disclose your age only if so”, e.g. so no PII about minors would be collected.

Discussed with @OBrezhniev in Slack, who said:

Interesting use case. Should be possible to implement without changes to circuits.

@ottomorac
Copy link
Member

Interesting. I will add to our confluence requirements tracking. Would you agree that we call this "conditional selective disclosure"? @nedgar ?

@nedgar
Copy link
Author

nedgar commented May 15, 2023

@otto-mora Hm, I'm not sure. I guess the naming could distinguish between the "pure" selective disclosure case, where the original value is unknown and no comparison is given (so holder generates proof using $eq on the current value) vs one where a comparison is given (as suggested above). It's not that different from using $noop operator, or $neq with some impossible value, or $nin with an empty set. Maybe it should just be made an option for selective disclosure, and if no condition is given it defaults to $eq with current value. If it's important to distinguish between these two cases, then "selective disclosure" and "conditional selective disclosure" make sense.

@github-actions
Copy link

This issue is stale because it has been open for 14 days with no activity.

@github-actions github-actions bot added the stale label May 30, 2023
@ottomorac ottomorac added enhancement New feature or request and removed stale labels May 30, 2023
@github-actions
Copy link

This issue is stale because it has been open for 14 days with no activity.

@github-actions github-actions bot added the stale label Jun 14, 2023
@ottomorac ottomorac removed the stale label Jun 15, 2023
@github-actions
Copy link

This issue is stale because it has been open for 14 days with no activity.

@github-actions github-actions bot added the stale label Jun 30, 2023
@nedgar
Copy link
Author

nedgar commented Jul 10, 2023

.

@ottomorac ottomorac removed the stale label Jul 10, 2023
@ottomorac
Copy link
Member

@nedgar let me look to see how we can prevent this stuff from being marked as stale. I believe there is a setting to add a special label. @mmilenkovic can you please try to see how we can add this setting https://github.com/actions/stale#exempt-issue-labels. Perhaps issues labeled as "bug", and "enhancement" can be exempt from the Github Actions bot?

@Kolezhniuk
Copy link
Collaborator

@demonsh

@github-actions
Copy link

This issue is stale because it has been open for 14 days with no activity.

@github-actions github-actions bot added the stale label Jul 27, 2023
@nedgar
Copy link
Author

nedgar commented Jul 27, 2023 via email

@nedgar
Copy link
Author

nedgar commented Jul 27, 2023

Perhaps issues labeled as "bug", and "enhancement" can be exempt from the Github Actions bot?

+1 for this.

@ottomorac ottomorac removed the stale label Jul 27, 2023
@vmidyllic
Copy link
Collaborator

Added enhancement to exclude stale actions.
For now, the simple workaround for this enhancement is to have several requests in the query, implementation of new behaviour for default operator for selective disclosure is not planned for next quarter.

@vmidyllic vmidyllic reopened this Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants