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

update changelog #450

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions openid-4-verifiable-credential-issuance-1_0.md
Original file line number Diff line number Diff line change
Expand Up @@ -2284,7 +2284,7 @@ be used):
`false`, the claim is not required to be included in the Credential. If the
`mandatory` parameter is omitted, the default value is `false`.

The rules defined in (##claims-description-processing) apply.
The rules defined in (#claims-description-processing) apply.

## Claims Description for Issuer Metadata {#claims-description-issuer-metadata}

Expand All @@ -2311,7 +2311,7 @@ following keys can be used to describe the claim or claims:
represented as language tag values defined in BCP47 [@!RFC5646]. There
MUST be only one object for each language identifier.

The rules defined in (##claims-description-processing) apply.
The rules defined in (#claims-description-processing) apply.

## Processing Rules for Claims Description Objects {#claims-description-processing}

Expand Down Expand Up @@ -2769,13 +2769,15 @@ The technology described in this specification was made available from contribut
* deferred credential response always returns an array (same as credential response)
* notification_id is now used for an issuance flow that can contain more than one credential
* Fixed #375: Enabled non-breaking extensibility
* removes `c_nonce` and `c_nonce_expires_in` from the Credential Error Response
* removes `c_nonce` and `c_nonce_expires_in` from the Credential Response
* Fixed #239: Completed IANA Considerations section
* add key attestation as additional information in a proof of possession and new proof type
* Change the syntax of credential metadata to use claims path queries
* change credential format identifier `vc+sd-jwt` to `dc+sd-jwt` to align with the media type in draft -06 of [@I-D.ietf-oauth-sd-jwt-vc] and update `typ` accordingly in examples
* use claims path pointer for mdoc based credentials
* removes `c_nonce_expires_in` from Nonce Endpoint
* removed `c_nonce_expires_in` from Nonce Endpoint
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a very minor comment:

Nonce Endpoint was introduced in d15 (initially with c_nonce_expires_in but at the end it was removed).
Perhaps, there is no need to include this line ("removed c_nonce_expired_in from Nonce Endpoint").

* removed `c_nonce` and `c_nonce_expires_in` from the token endpoint response
* added a Nonce Endpoint where a Client can acquire a fresh c_nonce value without the overhead of a full Credential Request

-14

Expand All @@ -2792,8 +2794,6 @@ The technology described in this specification was made available from contribut
* Define Credential Dataset as a term
* Define Credential Configuration as a term
* remove use of the `authorization_pending` and `slow_down` error codes
* removed `c_nonce` and `c_nonce_expires_in` from the token endpoint response
* added a Nonce Endpoint where a Client can acquire a fresh c_nonce value without the overhead of a full Credential Request

-13

Expand Down
Loading