Skip to content

Inline key presentation - #2

Merged
jricher merged 9 commits into
mainfrom
inline-key
Jul 27, 2026
Merged

jricher merged 9 commits into
mainfrom
inline-key

Conversation

@jricher

@jricher jricher commented Jul 25, 2026

Copy link
Copy Markdown
Owner
  • fix lint
  • Extract generatable examples into include files
  • Add --runtime-key to embed public key as signature parameters
  • Add example regeneration script and regenerate examples
  • Update verification examples.
  • clean up validation section

@c2bo c2bo left a comment

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 believe there is some confusion with kid vs keyid, although i am not sure i got everything right with the proposed changes.

Comment thread draft-richer-oauth-httpsig.md Outdated
Comment thread draft-richer-oauth-httpsig.md Outdated
Comment thread draft-richer-oauth-httpsig.md Outdated
Comment thread draft-richer-oauth-httpsig.md Outdated
Comment thread draft-richer-oauth-httpsig.md Outdated
Comment on lines +110 to +111
* Present its public key in the as described in {{embed-keys}}.
* Include the `kid` signature parameter uniquely identifying the key.

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 think the draft text should mandate that only either of them MUST be present

Co-authored-by: Christian Bormann <chris.bormann@gmx.de>
jricher and others added 2 commits July 27, 2026 06:37
Co-authored-by: Christian Bormann <chris.bormann@gmx.de>
Co-authored-by: Paul Bastian <paul.bastian@posteo.de>
@jricher
jricher merged commit 4e24cee into main Jul 27, 2026
2 checks passed
@jricher
jricher deleted the inline-key branch July 27, 2026 10:39
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.

4 participants