search

peppelinux / draft-demarco-oauth-status-assertions

OAuth 2.0 Status Assertions for Digital Credentials
Other
4 stars 5 forks source link

credential_pop typ with status-attestation-request+cwt + huge editorials #37

Closed SaraConsoliACN closed 4 months ago

SaraConsoliACN commented 4 months ago

this commit may resolve issue #20.

peppelinux commented 4 months ago

This PR is ready to be merged, a further PR will be create to address @marinaado 's comment https://github.com/peppelinux/draft-demarco-oauth-status-attestations/pull/45#issuecomment-2117623296 and the table related to the errors where the placeholder is here defined: https://github.com/peppelinux/draft-demarco-oauth-status-attestations/pull/37/files#diff-6018989306b8996e79d471eb123fc3b59796d1635384b6faac8cdb36cd5f49ccR472

regarding the first comment, since it is required that the status assertion issuer always respond with an application/json containing the status_assertion_errors, I'd try to use a general approach without defining specialized error section for both the requests and the responses