Skip to content

Bump dependencies - #330

Open
avandecreme wants to merge 6 commits into
eclipse-biscuit:mainfrom
avandecreme:bumps
Open

Bump dependencies#330
avandecreme wants to merge 6 commits into
eclipse-biscuit:mainfrom
avandecreme:bumps

Conversation

@avandecreme

Copy link
Copy Markdown

I couldn't bump rand_core and rand because they are tied to the version used by ed25519-dalek.

Comment thread biscuit-auth/src/error.rs
Comment on lines -93 to +96
InvalidLength,
InvalidLength(usize),
InvalidLastSymbol(usize, u8),
InvalidPadding,

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This is a breaking change

(None, _) => {
(Err(_), _) => {
return Err(error::Format::DeserializationError(
"deserialization error: unary operation is empty".to_string(),

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Not sure about that error message

(Ok(op_binary::Kind::TryOr), None) => Op::Binary(Binary::TryOr),
(Err(_), _) => {
return Err(error::Format::DeserializationError(
"deserialization error: binary operation is empty".to_string(),

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Same

Antoine Vandecrème added 2 commits May 15, 2026 11:46
@avandecreme
avandecreme force-pushed the bumps branch 2 times, most recently from 349847c to 217a244 Compare May 15, 2026 09:51
Comment thread biscuit-quote/Cargo.toml
proc-macro2 = "1"
quote = "1.0.14"
syn = { version = "1.0.85", features = ["full", "extra-traits"] }
syn = { version = "2.0.117", default-features = false }

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

syn 3 was released recently, I guess we could skip version 2

@avandecreme

Copy link
Copy Markdown
Author

@divarvel any interest in this?

@ixcans

ixcans commented Aug 6, 2026

Copy link
Copy Markdown

Hi @avandecreme Thanks for the PR!

The blocker in the description looks like it has cleared since this was opened in May: ed25519-dalek 3.0.0 went stable on 2026-07-06 and takes rand_core ^0.10, so rand/rand_core are no longer pinned back by it.

For what it's worth, the rest of the stack this PR would need has landed as stable releases too, all after this PR was opened:

crate stable version released
signature 3.0.0 2026-05-02
p256 0.14.0 2026-07-03
curve25519-dalek 5.0.0 2026-07-06
ed25519-dalek 3.0.0 2026-07-06

That may also unblock #312, its description notes there was no final RustCrypto release at the time, and that is no longer the case.

A downstream data point in case it helps prioritise: we build services on biscuit-auth, and because our own code shares SigningKey / PublicKey types with the library across an API boundary, we can't dual-version the crypto stack. So biscuit-auth 6.0.0's requirements transitively hold our whole workspace at ed25519-dalek 2, p256 0.13, rand 0.8, rand_core 0.6 and getrandom 0.2, we attempted the upgrade this week and had to back all of it out, while every other dependency moved.

Hi @divarvel , when you get a chance, could you please give a review to this PR? Thank you in advance!

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