Skip to content
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

remove previous_key from ThirdPartyBlockRequest #247

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

divarvel
Copy link
Collaborator

Same as for legacyPublicKeys, it is now deprecated and should not be set. It is kept in the schema to allow implementations to make sure it is not set.

Copy link

codspeed-hq bot commented Nov 21, 2024

CodSpeed Performance Report

Merging #247 will not alter performance

Comparing remove-previous-key-from-3rd-party-request (6ad363f) with v5 (fe3e74d)

Summary

✅ 12 untouched benchmarks

Copy link

codecov bot commented Nov 21, 2024

Codecov Report

Attention: Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.

Project coverage is 65.57%. Comparing base (fe3e74d) to head (6ad363f).
Report is 3 commits behind head on v5.

Files with missing lines Patch % Lines
biscuit-auth/src/token/third_party.rs 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##               v5     #247      +/-   ##
==========================================
+ Coverage   65.19%   65.57%   +0.38%     
==========================================
  Files          27       27              
  Lines        6981     6987       +6     
==========================================
+ Hits         4551     4582      +31     
+ Misses       2430     2405      -25     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@divarvel divarvel force-pushed the remove-previous-key-from-3rd-party-request branch from 8dada64 to e3c320f Compare November 21, 2024 15:43
Same as for `legacyPublicKeys`, it is now deprecated and should not be set. It is kept in the schema to allow implementations to make sure it is not set.
this exposes a new public method on `Biscuit` and `UnverifiedBiscuit` to get a block’s version, same as printing a block contents or getting its external key.

Exposing it on `UnverifiedBiscuit` will be useful for biscuit-cli
@divarvel divarvel force-pushed the remove-previous-key-from-3rd-party-request branch from d5f7b63 to 6ad363f Compare November 21, 2024 16:03
@divarvel divarvel requested a review from Geal November 22, 2024 12:57
@divarvel divarvel merged commit 6ad6f12 into v5 Nov 25, 2024
7 checks passed
@divarvel divarvel deleted the remove-previous-key-from-3rd-party-request branch November 25, 2024 07:50
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.

2 participants