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

Remove chrono #211

Closed
wants to merge 3 commits into from
Closed

Remove chrono #211

wants to merge 3 commits into from

Conversation

Adam-Gleave
Copy link
Contributor

Since RUSTSEC has been flagging up the vulnerabilities in chrono and the older time versions recently, our cargo audit workflow is grumbling about things quite a bit.

simple_asn1 has been updated to remove dependencies on chrono in favour of the time crate (which has had major feature updates), so I have updated that to the next minor version. I've also removed chrono from dev-dependencies and reworked the example to use time.

cargo audit doesn't complain about vulnerabilities any more.

@Keats
Copy link
Owner

Keats commented Oct 28, 2021

Can you do the PR on #160 instead?

@Adam-Gleave Adam-Gleave mentioned this pull request Oct 29, 2021
@Adam-Gleave
Copy link
Contributor Author

Sure! I've opened #213 now, I'll close this one 👍

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