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

updates as per ITI F2F meeting on 6 may #11

Merged
merged 2 commits into from
May 8, 2024

Conversation

ritikarawlani
Copy link
Contributor

Closes #

πŸ“‘ Description

βœ… Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed
  • I have selected a committee co-chair to review the PR

β„Ή Additional Information

ch-5.10.html Outdated
@@ -192,7 +169,7 @@ <h4 id="5.10.2.3">5.10.2.3 Payload</h4>
<ul><li class="bullet-list1">See the Payload sections within Detached (<a href="5.10.3.3">section 5.10.3.3</a>) and within Enveloping (<a href="5.10.4.3">section 5.10.4.3</a>) for specific guidance</li></ul>
<h4 id="5.10.2.4">5.10.2.4 Signature</h4>

<ul><li class="bullet-list1">The Signature SHALL at least support the hashing algorithm SHA256. The signature validation policy will apply to the signature and determine the acceptability</li>
<ul><li class="bullet-list1">The Signature SHALL use the algorithm defined in the <a href="5.10.2.1.1">5.10.2.1.1</a> alg parameter section above.</li>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

in the alg parameter

ch-5.10.html Outdated
<h5 id="5.10.2.1.2">5.10.2.1.2 "crit" (Critical) Parameter</h5>
<h5 id="5.10.2.1.2">5.10.2.1.2 "kid" (key identifier) Parameter</h5>
<ul>
<li>The kid parameter, if present, is a hint indicating which key was used to secure the JWS. The kid claim is used by Consumers for looking up the public key</li>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

"DSG Content Consumer"

looking up the public key for verification of digital signatures.

ch-5.10.html Outdated
@@ -126,50 +126,27 @@ <h5 id ="5.10.2.1.1">5.10.2.1.1 "alg" (Algorithm) Header Parameter</h5>
<ul>
<li class="bullet-list1">SHALL be present</li>
<li> It is recommend to use algorithms as specified in <a href="https://www.rfc-editor.org/rfc/rfc7518.html#section-3.1">RFC 7518</a> and <a href="https://www.etsi.org/deliver/etsi_ts/119300_119399/119312/01.01.01_60/ts_119312v010101p.pdf">ETSI TS 119 312</a></li>
<li>The use of RS256 SHALL be required for the purpose of interoperability testing. However, implementors SHOULD take into account additional considerations such as jurisdictional policies, quantum safe computing, and evolving guidance from RFC 7518 and ETSI TS 119 312.</li>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

RS256 algorithm SHALL be implemented

@ritikarawlani ritikarawlani merged commit 55913e6 into IHE:main May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant