Skip to content

fix(jose): prevent public/unprotected header overwriting protected header#809

Merged
azmeuk merged 2 commits into
mainfrom
fix-jose-337
Aug 27, 2025
Merged

fix(jose): prevent public/unprotected header overwriting protected header#809
azmeuk merged 2 commits into
mainfrom
fix-jose-337

Conversation

@lepture

@lepture lepture commented Aug 27, 2025

Copy link
Copy Markdown
Member

Fixes #337

Checklist

  • You ran the linters with prek.
  • You wrote unit test to demonstrate the bug you are fixing, or to stress the feature you are bringing.
  • You reached 100% of code coverage on the code you edited, without abusive use of pragma: no cover
  • If this PR is about a new feature, or a behavior change, you have updated the documentation accordingly.

  • You consent that the copyright of your pull request source code belongs to Authlib's author.

@lepture
lepture requested a review from azmeuk August 27, 2025 07:07
@sonarqubecloud

Copy link
Copy Markdown

@azmeuk azmeuk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@azmeuk
azmeuk merged commit 36b1246 into main Aug 27, 2025
18 checks passed
@azmeuk
azmeuk deleted the fix-jose-337 branch August 27, 2025 07:32
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.

JWS JSON integrity protected headers are owerwritten by unprotected headers

2 participants