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

sec: Remove vulnerable time-0.1.x chrono dependency #884

Closed
wants to merge 1 commit into from

Conversation

ixti
Copy link
Contributor

@ixti ixti commented Dec 16, 2022

This dependency is optional for chrono and enabled by default for backward compatibility only.

See: https://rustsec.org/advisories/RUSTSEC-2020-0071
See: https://github.com/chronotope/chrono/blob/v0.4.23/CHANGELOG.md#0416

@changeset-bot
Copy link

changeset-bot bot commented Dec 16, 2022

🦋 Changeset detected

Latest commit: 695e288

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
fnm Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Dec 21, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fnm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 12, 2024 4:13am

@ixti
Copy link
Contributor Author

ixti commented Jun 19, 2023

@Schniz is this something you can think might be merged, or should I close this PR and stop checking it?

@Schniz
Copy link
Owner

Schniz commented Jun 22, 2023

@Schniz is this something you can think might be merged, or should I close this PR and stop checking it?

In trying to rerun the tests but I’m afk (parental leave) so I’m not sure why it failed. We can ignore clippy

@ixti
Copy link
Contributor Author

ixti commented Jun 22, 2023

In trying to rerun the tests but I’m afk (parental leave) so I’m not sure why it failed. We can ignore clippy

Oh, congrats!!! Will try to make build green then meanwhile! :D

@Schniz
Copy link
Owner

Schniz commented May 27, 2024

addressed in another PR, thanks for the contribution 🙏

@Schniz Schniz closed this May 27, 2024
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.

2 participants