go-cose icon indicating copy to clipboard operation
go-cose copied to clipboard

headers: validate CWT claims

Open pranjalkole opened this issue 10 months ago • 4 comments

We could also refactor this to a validateCWTClaims function in cwt.go

pranjalkole avatar Jan 20 '25 21:01 pranjalkole

@veraison/go-cose-maintainers, can we get some 👀's on this?

SteveLasker avatar Jan 31 '25 16:01 SteveLasker

Codecov Report

Attention: Patch coverage is 3.63636% with 53 lines in your changes missing coverage. Please review.

Project coverage is 89.18%. Comparing base (eb9cdec) to head (c88a9da). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
headers.go 3.63% 53 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
- Coverage   91.12%   89.18%   -1.95%     
==========================================
  Files          12       12              
  Lines        2006     2053      +47     
==========================================
+ Hits         1828     1831       +3     
- Misses        122      168      +46     
+ Partials       56       54       -2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Feb 08 '25 08:02 codecov[bot]

@shizhMSFT, @qmuntal, @thomas-fossati can you PTAL?

SteveLasker avatar Apr 11 '25 15:04 SteveLasker

@pranjalkole can you please take a look at above changes?

SteveLasker avatar Nov 07 '25 06:11 SteveLasker