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

chore: update dependencies #89

Merged
merged 1 commit into from
Oct 21, 2024
Merged

Conversation

NULLx76
Copy link
Contributor

@NULLx76 NULLx76 commented Oct 20, 2024

Updating the dependencies only required a small change with regard to the base64 crate.
Everything seems to compile fine, doing cargo run errors with a 404 however but this also seems to happen for me on master.

It might also be nice to switch reqwest over from OpenSSL to the RusTLS, just so it doesn't require openssl to be installed to build. But didn't want to just lump it in without asking.

Finally, we still depend on hyper_old_types because we're using the typed headers for Link. Unfortunately these have not been added to headers yet: hyperium/headers#67, hyperium/headers#113. I think it's fine like this for now, but we should probably switch when those do land.

Fixes #87

@MarcoIeni MarcoIeni merged commit c9eb213 into rust-lang:master Oct 21, 2024
1 check passed
@MarcoIeni
Copy link
Member

It might also be nice to switch reqwest over from OpenSSL to the RusTLS, just so it doesn't require openssl to be installed to build. But didn't want to just lump it in without asking.

Makes sense to me if you want to do it in another PR 👍

@NULLx76 NULLx76 deleted the update_deps branch October 21, 2024 14:54
@NULLx76 NULLx76 mentioned this pull request Oct 21, 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.

Update dependencies
2 participants