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

deps: update openssl for RUSTSEC-2024-0357 #132

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

mihirn
Copy link
Contributor

@mihirn mihirn commented Jul 25, 2024

No description provided.

@mihirn mihirn requested a review from brayniac July 25, 2024 22:09
@swlynch99
Copy link
Contributor

Looks like you've got two issues that you need to address:

  • You've updated the lockfile into a state where it needs some additional updates. Running cargo build and then committing the changes it makes to the lockfile should fix this.
  • Due to the update to rust 1.80 it looks like we've also hit Type inference regression on nightly-2024-05-20 rust-lang/rust#125319. You'll need to update the time crate to the latest patch release to fix this.

@mihirn
Copy link
Contributor Author

mihirn commented Jul 25, 2024

Ugh, thanks. time is pulled in from clocksource so I'm guessing the better fix is to fix it upstream rather than having an explicit dependency here?

@swlynch99
Copy link
Contributor

swlynch99 commented Jul 25, 2024

No. All you need to do is run cargo update --precise time. This is not an issue with clocksource (although clocksource will probably run into a similar issue).

@mihirn mihirn merged commit fdab344 into pelikan-io:main Jul 25, 2024
21 checks passed
@mihirn
Copy link
Contributor Author

mihirn commented Jul 25, 2024

Ok, that worked, thanks!

@mihirn mihirn deleted the msn/openssl branch July 25, 2024 23:59
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.

3 participants