Skip to content

hardening: guard against homoglyph keys during claims (un)marshaling - #191

Merged
pkazmierczak merged 1 commit into
mainfrom
b-jwt-glyphs
Jul 14, 2026
Merged

hardening: guard against homoglyph keys during claims (un)marshaling#191
pkazmierczak merged 1 commit into
mainfrom
b-jwt-glyphs

Conversation

@pkazmierczak

@pkazmierczak pkazmierczak commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

This changeset introduces a small hardening of the library against JWT "claims
confusion" vulnerability, where a Unicode homoglyph of a registered claim name
could make struct-based validation and the returned map disagree.

Internal ref: https://hashicorp.atlassian.net/browse/SECVULN-47712

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

@pkazmierczak
pkazmierczak requested a review from a team as a code owner July 3, 2026 14:55

@sarahashi1337 sarahashi1337 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@pkazmierczak
pkazmierczak merged commit cf52420 into main Jul 14, 2026
7 checks passed
@pkazmierczak
pkazmierczak deleted the b-jwt-glyphs branch July 14, 2026 14:43
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.

3 participants