Skip to content

Conversation

@awoie
Copy link
Contributor

@awoie awoie commented Feb 12, 2024

This PR does:

This PR is non-breaking.


`access_denied`:

- The Wallet did not have the requested Credentials to satisfy the Authorization Request.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this really access_denied..?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What other error would you suggest? I would not create a dedicated error for privacy reasons.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, i don't really know an alternative, access_denied feels good enough after the WG discussion.

Co-authored-by: Kristina <52878547+Sakurann@users.noreply.github.com>
@Sakurann Sakurann merged commit 2629e38 into main Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Need better error description and examples for more common cases

6 participants