Skip to content

Fix crash when verifying corrupted token #217

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 11, 2024
Merged

Conversation

ptoffy
Copy link
Member

@ptoffy ptoffy commented Nov 11, 2024

These changes are now available in 5.1.1

Fix #213

@ptoffy ptoffy requested review from gwynne and 0xTim November 11, 2024 14:46
@ptoffy ptoffy requested a review from gwynne November 11, 2024 15:54
Copy link

codecov bot commented Nov 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.52%. Comparing base (087031e) to head (85f1d34).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
- Coverage   83.68%   83.52%   -0.16%     
==========================================
  Files          56       56              
  Lines        1483     1493      +10     
==========================================
+ Hits         1241     1247       +6     
- Misses        242      246       +4     
Files with missing lines Coverage Δ
Sources/JWTKit/JWTParser.swift 93.47% <100.00%> (+1.81%) ⬆️

... and 1 file with indirect coverage changes

@ptoffy ptoffy requested a review from gwynne November 11, 2024 17:00
@ptoffy ptoffy force-pushed the fix/corrupted-token-crash branch from 85f1d34 to f6bf463 Compare November 11, 2024 17:05
@ptoffy ptoffy added the semver-patch Internal changes only label Nov 11, 2024
@ptoffy ptoffy merged commit 6f745e9 into main Nov 11, 2024
22 checks passed
@ptoffy ptoffy deleted the fix/corrupted-token-crash branch November 11, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-patch Internal changes only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash when verifying a corrupt token
2 participants