Skip to content

Conversation

@Naccomy
Copy link
Contributor

@Naccomy Naccomy commented Jul 3, 2025

Summary

Add methods to create and inspect third party blocks:

  • Biscuit::third_party_request for creating third party requests
  • ThirdPartyRequest::create_block for creating third party blocks
  • Biscuit::append_third_party for creating biscuits with third party blocks
  • Biscuit::external_key for retrieving external keys from blocks
  • Add unit tests covering third party block functionality

Add methods to create and inspect third party blocks:
- `Biscuit::append_third_party` for creating biscuits with third party
  blocks
- `Biscuit::external_key` for retrieving external keys from blocks
- Add unit tests covering third party block functionality
@Naccomy Naccomy mentioned this pull request Jul 3, 2025
@divarvel
Copy link
Contributor

divarvel commented Jul 3, 2025

Neat! A good addition would be to exposing support for requests, since in most cases, we don’t want to transmit the whole token to the third-party.

@divarvel
Copy link
Contributor

divarvel commented Jul 3, 2025

You’ll need to sign the Eclipse Contributor Agreement: https://github.com/eclipse-biscuit/biscuit-rust/blob/main/CONTRIBUTING.md#eclipse-contributor-agreement

@Naccomy
Copy link
Contributor Author

Naccomy commented Jul 3, 2025

Neat! A good addition would be to exposing support for requests, since in most cases, we don’t want to transmit the whole token to the third-party.

I’ve updated the code to include ThirdPartyRequest and ThirdPartyBlock. The Biscuit::append_third_party function is now implemented to match the behaviour of the Rust version.

@divarvel
Copy link
Contributor

I’m sorry, I missed your answer. This looks great!

@divarvel
Copy link
Contributor

I’ll add pyi bindings myself.

@divarvel divarvel merged commit 3842026 into eclipse-biscuit:main Sep 19, 2025
1 check passed
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