Skip to content

Conversation

@tarcieri
Copy link
Contributor

Most of the changes in this PR are actually from the associated rand crate updates which are happening in rust-random/rand#1697, notably OsRng has been renamed to SysRng (and is now provided by the getrandom crate).

We do use some rand_core APIs in a few places though, like the STROBE implementation, where this migrates from rand_core::le to rand_core::utils.

Most of the changes in this PR are actually from the associated `rand`
crate updates which are happening in rust-random/rand#1697, notably
`OsRng` has been renamed to `SysRng` (and is now provided by the
`getrandom` crate).

We do use some `rand_core` APIs in a few places though, like the STROBE
implementation, where this migrates from `rand_core::le` to
`rand_core::utils`.
Comment on lines +18 to +20
# https://github.com/rust-random/rand/pull/1697
rand = { git = "https://github.com/rust-random/rand", rev = "99ae06fb348e5c5fee0fe41d2201bd4cbb107635" }
getrandom = { git = "https://github.com/rust-random/getrandom.git", rev = "f67c70110f706f80d7dd6fa52e38d0920754fe2c" }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could go either way on waiting for these to be merged. Right now the build is broken on main, so I feel like getting it fixed would probably be for the best, even if we temporarily have some patches to do it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Agree

@tarcieri tarcieri requested a review from rozbb December 27, 2025 16:21
Copy link
Contributor

@rozbb rozbb left a comment

Choose a reason for hiding this comment

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

Thank you for catching this!

@tarcieri tarcieri merged commit 9d6ec94 into main Dec 27, 2025
52 checks passed
@pinkforest
Copy link
Contributor

pinkforest commented Dec 28, 2025

Needs a release still, getting some conflicts (through aead's) between pinned rc's.

carloskiki pushed a commit to carloskiki/curve25519-dalek that referenced this pull request Jan 3, 2026
Most of the changes in this PR are actually from the associated `rand`
crate updates which are happening in rust-random/rand#1697, notably
`OsRng` has been renamed to `SysRng` (and is now provided by the
`getrandom` crate).

We do use some `rand_core` APIs in a few places though, like the STROBE
implementation, where this migrates from `rand_core::le` to
`rand_core::utils`.
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.

4 participants