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 notice ed25519-dalek #1361

Closed

Conversation

pinkforest
Copy link
Contributor

@pinkforest pinkforest commented Aug 14, 2022

Closes #1360

NOTE: This does NOT necessarily mean the crypto on ed25519-dalek is inherently broken or insecure as of now

e.g. Depending on how we classify / see broken / insecure - people often see crypto-failure where pub API was not misused

dalek-cryptography/ed25519-dalek#192
dalek-cryptography/ed25519-dalek#209
dalek-cryptography/ed25519-dalek#205

EDIT: This is about older versions having pub API weakness which needs a notice. Not a maintenance issue at this stage.

Will Merge when the new release is merged which is still some time away.

Signed-off-by: pinkforest <36498018+pinkforest@users.noreply.github.com>
Signed-off-by: pinkforest <36498018+pinkforest@users.noreply.github.com>
@pinkforest pinkforest changed the title Add advisory ed25519-dalek Add advisory ed25519-dalek Aug 14, 2022
@pinkforest pinkforest added crypto Crypto lib notice Unmaintained Informational / Unmaintained labels Aug 14, 2022
@pinkforest pinkforest changed the title Add advisory ed25519-dalek Add notice ed25519-dalek Aug 14, 2022
@Shnatsel
Copy link
Member

Shnatsel commented Dec 8, 2022

The associated issue has been closed, so I'm going to go ahead and close this as well.

However, the fix still hasn't shipped to crates.io; if it turns out that crates.io releases are infeasible, we might need to revive this PR.

@Shnatsel Shnatsel closed this Dec 8, 2022
@pinkforest pinkforest reopened this Dec 9, 2022
@pinkforest
Copy link
Contributor Author

pinkforest commented Dec 9, 2022

We still need to notify about older versions re: PK Oracle -

I've re-opened and further clarified the tracking issue that reflected this.

I will edit the PR in a moment to remove the unmaintained part of it.

@pinkforest pinkforest removed the Unmaintained Informational / Unmaintained label Dec 9, 2022
@pinkforest pinkforest closed this Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crypto Crypto lib notice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ed25519-dalek PK Oracle
2 participants