-
Notifications
You must be signed in to change notification settings - Fork 15
Allow other Wallet Attestation formats #291
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
Conversation
jogu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There seems to be a git conflict in the normative changes unfortunately - would be good if you can resolve that too please.
Co-authored-by: Joseph Heenan <joseph@authlete.com>
Co-authored-by: Joseph Heenan <joseph@authlete.com>
4c98820 to
6b3fed3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some editorial comments and the following two normative comments:
- use "if applicable" when talking about PAR requirements
- exclude trust anchor from x5c
Co-authored-by: Oliver Terbu <oliver.terbu@mattr.global> Co-authored-by: Joseph Heenan <joseph@authlete.com>
awoie
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
had one minor nit
Co-authored-by: Oliver Terbu <oliver.terbu@mattr.global>
Proposal making the wallet attestations format recommended instead of mandatory. Happy to change the text if we get consensus on the wg for a different path, but this seemed to be the current outcome?
closes #211