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

Pin package version of reqwest-tracing to fix build error #112

Closed
wants to merge 1 commit into from

Conversation

recursiveforte
Copy link

@recursiveforte recursiveforte commented Nov 13, 2024

reqwest-tracing recently had an update from 0.5.3 to 0.5.4, which broke builds for this project. I'm pinning the dependency version to allow it to compile for now.

To reproduce, run cargo update to update dependencies in the cargo.lock, then cargo build.

@rgreinho
Copy link
Owner

Hi there! thanks a lot for reporting the issue.

However pinning the package is not the solution to the problem. The problem is that we need to update reqwest-tracing, reqwest-retry, and reqwest-middleware at the same time.

I'll submit a patch soon, and I'll release the next version.

@rgreinho rgreinho closed this Nov 14, 2024
@rgreinho
Copy link
Owner

The fix was applied in #113 and a new version (2.2.6) release with #114 .

@recursiveforte recursiveforte deleted the patch-1 branch November 17, 2024 20:27
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