Skip to content

Update chacha20: use ChaChaCore directly; remove bytes_until_reseed field#1726

Merged
dhardy merged 4 commits into
masterfrom
push-kzkqyroqyklv
Feb 7, 2026
Merged

Update chacha20: use ChaChaCore directly; remove bytes_until_reseed field#1726
dhardy merged 4 commits into
masterfrom
push-kzkqyroqyklv

Conversation

@dhardy

@dhardy dhardy commented Feb 2, 2026

Copy link
Copy Markdown
Member
  • Added a CHANGELOG.md entry

Depends on RustCrypto/stream-ciphers#516.

@dhardy dhardy requested review from newpavlov and tarcieri February 2, 2026 08:59
@dhardy

dhardy commented Feb 2, 2026

Copy link
Copy Markdown
Member Author

See also this comment: #1686 (comment)

Comment thread src/rngs/thread.rs Outdated
@dhardy

dhardy commented Feb 2, 2026

Copy link
Copy Markdown
Member Author

@newpavlov since this is still blocked by a new chacha20 release can you also review #1727? I'll rebase this after.

@dhardy dhardy force-pushed the push-kzkqyroqyklv branch 3 times, most recently from 2b89cdd to 683fc27 Compare February 2, 2026 14:49
Comment thread src/rngs/thread.rs
Comment thread src/rngs/thread.rs Outdated
@dhardy dhardy force-pushed the push-kzkqyroqyklv branch 2 times, most recently from a738623 to 34373f0 Compare February 2, 2026 15:31
Comment thread src/rngs/thread.rs
@dhardy dhardy force-pushed the push-kzkqyroqyklv branch from 34373f0 to 03db311 Compare February 5, 2026 15:30
@dhardy dhardy marked this pull request as ready for review February 5, 2026 15:30
@dhardy

dhardy commented Feb 5, 2026

Copy link
Copy Markdown
Member Author

Updated.

I could put out a new rand RC with this, but I won't bother without a good reason. The chacha20 RC.11 did not bring breaking changes so the current rand RC should still be fine using that.

@tarcieri

tarcieri commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

@dhardy I can put out another RC with RustCrypto/stream-ciphers#529 though hopefully it didn't impact anything in rand?

@dhardy dhardy merged commit 28e3df8 into master Feb 7, 2026
16 checks passed
@dhardy dhardy deleted the push-kzkqyroqyklv branch February 7, 2026 08:08
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