Skip to content

Adding testvectors. Adding clarifications in the body.#24

Open
hannestschofenig wants to merge 2 commits into
mainfrom
testvectors
Open

Adding testvectors. Adding clarifications in the body.#24
hannestschofenig wants to merge 2 commits into
mainfrom
testvectors

Conversation

@hannestschofenig

@hannestschofenig hannestschofenig commented May 28, 2026

Copy link
Copy Markdown
Collaborator

In this PR I am making three changes:

  1. Adding Test Vectors for JOSE based on the implementation at Add experimental JOSE PQC KEM support panva/jose#870

  2. Clarifications regarding the private key encoding

  3. Clarifications regarding KDF usage

@hannestschofenig hannestschofenig requested a review from tireddy2 May 29, 2026 15:21
Comment on lines +449 to 457
The following entry is added to the "JSON Web Signature and Encryption Header Parameters" registry:

- Header Parameter Name: ek
- Header Parameter Description: KEM ciphertext
- Header Parameter Usage Location(s): JWE
- Change Controller: IESG
- Specification Document(s): [[TBD: This RFC]]

The following entries are added to the "JSON Web Signature and Encryption Algorithms" registry:

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
The following entry is added to the "JSON Web Signature and Encryption Header Parameters" registry:
- Header Parameter Name: ek
- Header Parameter Description: KEM ciphertext
- Header Parameter Usage Location(s): JWE
- Change Controller: IESG
- Specification Document(s): [[TBD: This RFC]]
The following entries are added to the "JSON Web Signature and Encryption Algorithms" registry:

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Removed it because it will be registered via JOSE HPKE first. The description of the JOSE HPKE-defined parameter does not quite fit but this is something we will have to look into.

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.

1 participant