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

RUSTSEC-2024-0320: yaml-rust is unmaintained #84

Closed
andreacfromtheapp opened this issue Jul 6, 2024 · 4 comments
Closed

RUSTSEC-2024-0320: yaml-rust is unmaintained #84

andreacfromtheapp opened this issue Jul 6, 2024 · 4 comments

Comments

@andreacfromtheapp
Copy link

andreacfromtheapp commented Jul 6, 2024

Hi,

I am learning Rust (very early stages still) and, along with it, computer science. Because I want to be a good self-taught software engineer. I recently started exploring and learning about AppSec/DevSec and the importance of Securing the entire SDLC. Among other "findings" (tools, websites, guidelines and so forth), I just found out about RustSec.

Because I love learning by both studying and doing, I have an itch to scratch app that I will (knock knock) start coding as soon as I finish The Book and Rust in Action. That said, I have already researched the tools and crates I will be using. One of them is rusti18n. So today, out of curiosity I ran cargo audit (from RustSec) and found out about https://rustsec.org/advisories/RUSTSEC-2024-0320 impacting this very project.

I believe it would be beneficial if rusti18n could adopt the proposed resolution of that CVE: switching to a maintained package.

Please and thank you

EDIT post merge:

I notified that this project has fixed the issue on the CVE by linking it here. I renamed the issue as well, as everyone seem to use the same one. cheers and thank you!

@andreacfromtheapp
Copy link
Author

here's the full output of cargo audit to put things into a dependency tree:

╰─❯ cargo audit
    Fetching advisory database from `https://github.com/RustSec/advisory-db.git`
      Loaded 630 security advisories (from /Users/andreacfromtheapp/.cargo/advisory-db)
    Updating crates.io index
    Scanning Cargo.lock for vulnerabilities (458 crate dependencies)
Crate:     yaml-rust
Version:   0.4.5
Warning:   unmaintained
Title:     yaml-rust is unmaintained.
Date:      2024-03-20
ID:        RUSTSEC-2024-0320
URL:       https://rustsec.org/advisories/RUSTSEC-2024-0320
Dependency tree:
yaml-rust 0.4.5
└── serde_yaml 0.8.26
    ├── rust-i18n-support 3.0.1
    │   ├── rust-i18n-macro 3.0.0
    │   │   └── rust-i18n 3.0.1
    │   │       └── this_is_my_package 0.1.0
    │   └── rust-i18n 3.0.1
    └── rust-i18n-macro 3.0.0

warning: 1 allowed warning found

yaml-rust is being pulled in by serde_yaml which is archived and deprecated.

I hope this helps.

@Tanguille
Copy link

What would be even better in my opinion is to split this crate up into features. In a project I work on we don't even use yaml but toml instead so it would be amazing to have it split up in to features so you aren't affected by dependencies you don't even need.

@huacnlee
Copy link
Member

huacnlee commented Aug 9, 2024

image https://github.com//pull/86

@huacnlee huacnlee closed this as completed Aug 9, 2024
@andreacfromtheapp andreacfromtheapp changed the title CVE RUSTSEC-2024-0320 yaml-rust is unmaintained; Consider using a maintained alternative RUSTSEC-2024-0320: yaml-rust is unmaintained Aug 9, 2024
@andreacfromtheapp
Copy link
Author

I notified that this project has fixed the issue on the rustsec/advisory-db#1921 by linking it here. I renamed the issue as well, as everyone seem to use the same one. cheers and thank you!

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

No branches or pull requests

3 participants