Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add disclosure for hasbrown's borsh encoding #2100

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kayabaNerve
Copy link
Contributor

I wrote this as fixed in 0.16 due to that being the plan documented here: rust-lang/hashbrown#570 (comment)

Until that PR is merged and 0.16 is released, this cannot be merged/must be edited to be described as unpatched.

@clarfonthey
Copy link

There was a proposal to release the fix as 0.15.1 and just yank 0.15.0, so, might be worth verifying which of the two versions is going to be used for the fix.

It's unclear with what version it will be and when it will be. I'd prefer to make this valid and file-able now than risk someone starting to use this functionality (which will make removing it more complicated).
@kayabaNerve
Copy link
Contributor Author

kayabaNerve commented Oct 26, 2024

I've removed the patched notice so this can be filed now (warning other people off from adopting this functionality) while we continue to wait for a patched version.

@amousset amousset added the Waiting-Author Waiting on Author label Nov 10, 2024
@kayabaNerve
Copy link
Contributor Author

Sorry for my delay in updating this. It correctly notes when patched now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Waiting-Author Waiting on Author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants