-
Notifications
You must be signed in to change notification settings - Fork 4
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
Define entity_statement_signing_alg_values_supported metadata parameter #81
base: main
Are you sure you want to change the base?
Define entity_statement_signing_alg_values_supported metadata parameter #81
Conversation
JSON array containing a list of the JWS signing algorithms | ||
(<spanx style="verb">alg</spanx> values) | ||
supported by the Entity to sign Entity Statements. | ||
If omitted, the default value is <spanx style="verb">["RS256"]</spanx>. |
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.
On its own, the definition of this parameter is okay. However, having this parameter parameter defined would not be sufficient for the entities in a federation to be able to use a specific JWS algorithm when more than one is supported. People will ask - what do we do with this parameter? How do I get an ES signed with say ES256 when RS256 and ES256 are supported?
Considering that this parameter would be at the end informational only, I approve it for the scopes proposed. |
Last time when we discussed this (and I was present), there were two solutions. Both solutions have this metadata parameter, that lists the supported JWS algs for entity statements. The difference is:
Solution 1 seems simpler, in terms of specification and API. Solution 2 has the benefit that in multi anchored / lateral federations, where one of the federations supports additional JWS algs, but the algs still intersect (there is a commonly supported JWS alg, like RS256), consumers will be able to collect Entity Statements with an alg that they can work it. Otherwise they won't be able to do this. |
As discussed on today's Federation editors' call, we would want actionable guidance on how and when to use this parameter before including this feature in the specification. |
Fix #65
Cc: @Razumain