Skip to content

Use Hono JWT verification for Extensions assertions - #187

Merged
admdly merged 2 commits into
mainfrom
agent/extensions-assertion-hardening
Aug 5, 2026
Merged

Use Hono JWT verification for Extensions assertions#187
admdly merged 2 commits into
mainfrom
agent/extensions-assertion-hardening

Conversation

@admdly

@admdly admdly commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace the custom JWT parsing and Web Crypto verification with Hono's pinned HS256 JWT verifier.
  • Keep a small assertion-profile validator for the Extensions issuer, audience, purpose, version, subject, lifetime, and clock-skew rules.
  • Require integer NumericDate claims in the assertion profile.
  • Accept the current and previous signing secrets during rotation.
  • Document the rotation procedure and local previous-secret configuration.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 5, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
api 781d22c Commit Preview URL

Branch Preview URL
Aug 05 2026, 06:13 PM

@admdly admdly self-assigned this Aug 5, 2026
@admdly
admdly marked this pull request as ready for review August 5, 2026 17:37

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 4 files

Requires human review: Introduces acceptance of a previous signing secret for key rotation, altering authentication policy and requiring human review to verify the rotation scheme and security implications.

Re-trigger cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 2 files (changes from recent commits).

Requires human review: Replaces core authentication logic, adds a previous-secret rotation mechanism (ASSERTION_SIGNING_SECRET_PREVIOUS), and enforces stricter claim validation. These security and operational changes require human review to confirm correctness and assess risk.

Re-trigger cubic

@admdly
admdly merged commit 5153eaa into main Aug 5, 2026
9 checks passed
@admdly
admdly deleted the agent/extensions-assertion-hardening branch August 5, 2026 19:00
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