-
Notifications
You must be signed in to change notification settings - Fork 107
reqwest upgrade, using rusttls #2266
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
Conversation
|
Contract comparison - from fe875bc to 5b0c3be
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request attempts to upgrade the reqwest HTTP client library from version 0.11/0.12 to 0.13 and switch from native TLS to rustls for TLS implementation across multiple crates in the project.
Changes:
- Upgrade
reqwestdependency from 0.11/0.12 to 0.13.1 in four Cargo.toml files - Add
rustlsfeature and remove TLS-related features (default-tls, static-tls) in favor of rustls - Update Cargo.lock with corresponding dependency changes
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tools/git-scraper/Cargo.toml | Upgrades reqwest to 0.13 with rustls feature |
| sdk/http/Cargo.toml | Upgrades reqwest to 0.13, removes TLS feature flags, adds rustls |
| framework/snippets/Cargo.toml | Removes TLS-related feature flags that depended on sdk/http features |
| framework/meta/Cargo.toml | Upgrades reqwest to 0.13 with rustls feature |
| Cargo.lock | Reflects dependency changes including new rustls-related dependencies and removal of native-tls |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.