Skip to content

Clarified intent_to_retain value when not present#730

Open
Sakurann wants to merge 1 commit intomainfrom
intent-to-retain
Open

Clarified intent_to_retain value when not present#730
Sakurann wants to merge 1 commit intomainfrom
intent-to-retain

Conversation

@Sakurann
Copy link
Copy Markdown
Collaborator

@Sakurann Sakurann commented Apr 26, 2026

resolves #669 and potentially #642 - need to discuss if we want to add additional text to clarify usage of use is not mandatory

Copy link
Copy Markdown
Member

@selfissued selfissued left a comment

Choose a reason for hiding this comment

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

We should not be creating a default interpretation of not specifying this parameter in a way that requires changes to existing wallets.

Copy link
Copy Markdown
Collaborator

@jogu jogu left a comment

Choose a reason for hiding this comment

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

resolves #669 and potentially #642 -

I don't think this touches #642.


`intent_to_retain`
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5].
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5]. If not specified, the Wallet uses the value `false`.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I can't find any place where we actually reached a conclusion on what we should do.

#669 (comment) says:

explicitly stating that intent_to_retain being optional means that the verifier is not specifying if it will retain or not, which is arguably what the current spec says.

@brentzundel
Copy link
Copy Markdown
Collaborator

discussed in call today. Some suggested changes to the text were made.

@brentzundel
Copy link
Copy Markdown
Collaborator

Seemed to be consensus that if the verifier is not expressing anything, the wallet should indicate that the verifier hasn't expressed anything.


`intent_to_retain`
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5].
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5]. If not specified, the Wallet uses the value `false`.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5]. If not specified, the Wallet uses the value `false`.
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5]. If absent, the Verifier makes no statement on `IntentToRetain`.


`intent_to_retain`
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5].
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5]. If not specified, the Wallet uses the value `false`.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5]. If not specified, the Wallet uses the value `false`.
: OPTIONAL. A boolean that is equivalent to `IntentToRetain` variable defined in Section 8.3.2.1.2.1 of [@ISO.18013-5]. If absent, the Verifier makes no statement on `IntentToRetain`.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify intent_to_retain

5 participants