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

Updated dependencies #521

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

AlexSherbinin
Copy link

Updated dependencies, added rust-toolchain.toml file with nightly channel described.

Cargo.toml Outdated Show resolved Hide resolved
@S0c5
Copy link

S0c5 commented Feb 9, 2024

@Fishrock123 is there an interest in merging this PR? We're facing issues with nested dependencies that are outdated and are breaking our apps

@Fishrock123
Copy link
Member

Fishrock123 commented Feb 10, 2024

I will take a dependency upgrade that does not cause a major version (is not a breaking change for exposed interfaces)

I will take an update to tokio/hyper and request/curl only if those updates include it under a separate feature flag, with the dependency version in the flag, which was the big mistake here all along.

in more clear terms, the hyperium_http feature should have been hyperium_http_02 (the 0.2 version), so for hyper http 1.0 it should be hyperium_http_1

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.

None yet

4 participants