Skip to content

fix: pin ed25519-dalek to 2.1.1 and add cargo-audit to CI - #208

Merged
Calebux merged 2 commits into
zintarh:mainfrom
Suzan-max:77-audit-ed25519-dalek-version-for-known-vulnerabilities
Jul 29, 2026
Merged

fix: pin ed25519-dalek to 2.1.1 and add cargo-audit to CI#208
Calebux merged 2 commits into
zintarh:mainfrom
Suzan-max:77-audit-ed25519-dalek-version-for-known-vulnerabilities

Conversation

@Suzan-max

Copy link
Copy Markdown
Contributor

Audited dev-dependency ed25519-dalek against the RustSec advisory database: only known advisory is RUSTSEC-2022-0093 (double public key signing oracle), fixed in 2.0+ — the crate's current 2.1 requirement is unaffected. Pin to the exact latest 2.1.x patch (2.1.1) instead of a loose "2.1" range so patch upgrades are deliberate, and add a rustsec/audit-check job to CI so future advisories are caught automatically. Also checked soroban-sdk 21.7.1: no entry exists in the RustSec advisory-db, so there are no known CVEs against it.

Closes #77

Audited dev-dependency ed25519-dalek against the RustSec advisory
database: only known advisory is RUSTSEC-2022-0093 (double public key
signing oracle), fixed in 2.0+ — the crate's current 2.1 requirement is
unaffected. Pin to the exact latest 2.1.x patch (2.1.1) instead of a
loose "2.1" range so patch upgrades are deliberate, and add a
rustsec/audit-check job to CI so future advisories are caught
automatically. Also checked soroban-sdk 21.7.1: no entry exists in the
RustSec advisory-db, so there are no known CVEs against it.

Closes zintarh#77
@drips-wave

drips-wave Bot commented Jul 24, 2026

Copy link
Copy Markdown

@Suzan-max Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Calebux
Calebux merged commit 5189fc8 into zintarh:main Jul 29, 2026
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.

Audit ed25519_dalek version for known vulnerabilities

2 participants