Skip to content

Conversation

@dignifiedquire
Copy link
Member

@dignifiedquire dignifiedquire commented Sep 25, 2025

Needs some review & feedback for feature naming & usage of the new inout buffers

Closes #176

@dignifiedquire
Copy link
Member Author

Still debugging some failing tests currently

@dignifiedquire
Copy link
Member Author

Tests are mostly passing, no idea why crypto_secretstream wasm builds fail, they are passing locally..

@tarcieri
Copy link
Member

It looks like getrandom is being unconditionally enabled somehow

@dignifiedquire
Copy link
Member Author

@tarcieri I have discovered what I think is either a bug in the code I wrote, or a regression. Not sure yet. I have pushed a failing test here: 72eb980 any idea if there have been changes somewhere that could be the reason for this?

@tarcieri
Copy link
Member

@dignifiedquire I wonder if #177 might've caused that (i.e. adding clamping)

@dignifiedquire
Copy link
Member Author

@tarcieri yep, I hadn't seen that PR, that is exactly what breaks things, undoing that change makes things work again

@dignifiedquire
Copy link
Member Author

@tarcieri I have added a from_clamped method for now, which allows me to fix the issue, while preserving the security fixes from #177 for new.

@tarcieri
Copy link
Member

@dignifiedquire might bikeshed the name in a followup, but this is fine for now

@tarcieri tarcieri merged commit 742b605 into RustCrypto:master Sep 29, 2025
33 checks passed
@dignifiedquire
Copy link
Member Author

@tarcieri any chance I can get a pre-release of this?

@tarcieri
Copy link
Member

tarcieri commented Oct 2, 2025

which one(s) specifically?

@dignifiedquire
Copy link
Member Author

I just need crypto-box currently

github-merge-queue bot pushed a commit to n0-computer/iroh that referenced this pull request Oct 6, 2025
## Description

- Upgrade to [email protected]
- Upgrade dalek crates to prereleases


Blocked on 

- [x] RustCrypto/nacl-compat#182
- [x] pubky/mainline#71
- [x] pubky/pkarr#186

Closes #3198 

## Breaking Changes

- interfaces that depend on `rand` now need `0.9`

## Notes & open questions

<!-- Any notes, remarks or open questions you have to make about the PR.
-->
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.

rand 0.9.0 support

2 participants