-
Notifications
You must be signed in to change notification settings - Fork 19
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
Editorial change in Wallet Solution 'alg' HEADER #286
Editorial change in Wallet Solution 'alg' HEADER #286
Conversation
docs/en/wallet-solution.rst
Outdated
@@ -92,7 +92,7 @@ Header | |||
* - **Key** | |||
- **Value** | |||
* - alg | |||
- Algorithm used to verify the token signature. It MUST be one of the possibile values indicated `here <https://italia.github.io/eudi-wallet-it-docs/versione-corrente/en/algorithms.html>`_ (e.g., ES256). | |||
- Algorithm used to verify the token signature. It MUST be one of the possibile values indicated here [here](https://italia.github.io/eudi-wallet-it-docs/versione-corrente/en/algorithms.html) (e.g., ES256). |
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.
this was the RST href while the one your proposing here is the Markdown href type
change from markdown hyperlink form to RST hyperlink form
changed the hyperlink word from *here* to *table*
* - **Key** | ||
- **Value** | ||
* - alg | ||
- Algorithm used to verify the token signature. It MUST be one of the possibile values indicated in this `table <https://italia.github.io/eudi-wallet-it-docs/versione-corrente/en/algorithms.html>`_ (e.g., ES256). |
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.
- Algorithm used to verify the token signature. It MUST be one of the possibile values indicated in this `table <https://italia.github.io/eudi-wallet-it-docs/versione-corrente/en/algorithms.html>`_ (e.g., ES256). | |
- Algorithm used to verify the token signature. It MUST be one of the values included in this `table <https://italia.github.io/eudi-wallet-it-docs/versione-corrente/en/algorithms.html>`_ (e.g., ES256). |
docs/en/wallet-solution.rst
Outdated
* - alg | ||
- Algorithm used to verify the token signature. It MUST be one of the possibile values indicated in this `table <https://italia.github.io/eudi-wallet-it-docs/versione-corrente/en/algorithms.html>`_ (e.g., ES256). | ||
* - kid | ||
- Thumbprint of the public key used for signing. |
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.
- Thumbprint of the public key used for signing. | |
- Thumbprint of the public key used for signing, according to [RFC 7638](https://www.rfc-editor.org/rfc/rfc7638.html] |
please check that RFC 7638 is in the referenced standard and then use the rst ref pointing to standards.rst rfc 7638 instead of the markdown href
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.
Yes, RFC 7638 is present in docs/common/standards.rst. I'll edit the sentence as you requested
added RCF reference from standards.rst
This PR aims to solve issue #254
Editorial change in Wallet Solution 'alg' HEADER