Skip to content

workspace: bump RustCrypto dependencies to new set of releases - #312

Open
baloo wants to merge 1 commit into
eclipse-biscuit:mainfrom
baloo:baloo/bump-rustcrypto-pre-releases
Open

workspace: bump RustCrypto dependencies to new set of releases#312
baloo wants to merge 1 commit into
eclipse-biscuit:mainfrom
baloo:baloo/bump-rustcrypto-pre-releases

Conversation

@baloo

@baloo baloo commented Sep 15, 2025

Copy link
Copy Markdown

This is a very early attempt to update dependencies to use new releases of RustCrypto.

We (RustCrypto) don't have an final release yet, but this is meant to help test biscuit with code using pre-releases.

@divarvel

Copy link
Copy Markdown
Contributor

This looks good! the failing check is because of a change in an error message, i’ll just have to refresh the samples

@divarvel

Copy link
Copy Markdown
Contributor

The crypto API changes might have implications on https://github.com/eclipse-biscuit/biscuit-wasm

@baloo

baloo commented Sep 16, 2025

Copy link
Copy Markdown
Author

Changes to rand_core in 0.9 and getrandom definitely affect wasm runtime. Cryptography changes, I don't expect any effect.

I haven't used wasm in a little while, and I haven't looked yet at the changes required there to get an RNG.

baloo added a commit to baloo/biscuit-wasm that referenced this pull request Sep 22, 2025
pendant change to the changes made to biscuit-auth
(eclipse-biscuit/biscuit-rust#312)
baloo added a commit to baloo/biscuit-wasm that referenced this pull request Sep 22, 2025
pendant change to the changes made to biscuit-auth
(eclipse-biscuit/biscuit-rust#312)
@baloo
baloo force-pushed the baloo/bump-rustcrypto-pre-releases branch from 7c38a58 to 36e250a Compare September 22, 2025 05:37
baloo added a commit to baloo/biscuit-wasm that referenced this pull request Sep 22, 2025
pendant change to the changes made to biscuit-auth
(eclipse-biscuit/biscuit-rust#312)
@Geal

Geal commented Sep 22, 2025

Copy link
Copy Markdown
Contributor

Noting that this will change the public API, so we'll ship the changes as part of the next major release

@ixcans ixcans mentioned this pull request Aug 6, 2026
@divarvel

divarvel commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

I think an update on #330 should supersede this PR

@baloo baloo changed the title workspace: bump RustCrypto dependencies to pre-releases workspace: bump RustCrypto dependencies to new set of releases Aug 6, 2026
@baloo
baloo force-pushed the baloo/bump-rustcrypto-pre-releases branch from 36e250a to f49c55f Compare August 6, 2026 18:05
@baloo

baloo commented Aug 6, 2026

Copy link
Copy Markdown
Author

Bumped the commit to the new set of releases.

I've also removed the direct dependencies on ecdsa and elliptic-curve as they are only use through p256. p256 should forward the features as expected.

@baloo
baloo marked this pull request as ready for review August 6, 2026 18:06
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.

3 participants