deps: Update dependency reqwest to v0.13.3#551
Conversation
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughBumps the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
third_party/rust/deps.MODULE.bazel (1)
7-17: Update feature name to match reqwest v0.13.x breaking changes.The feature
rustls-tlswas renamed torustlsin reqwest v0.13.0. The build failure reported by Renovate confirms that reqwest 0.13.1 does not have therustls-tlsfeature, which is causing the cargo-bazel lockfile generation to fail.🔎 Fix: Update the feature name from rustls-tls to rustls
crate.spec( default_features = False, features = [ "blocking", - "rustls-tls", + "rustls", "http2", "charset", ], package = "reqwest", version = "=0.13.1", )
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
third_party/rust/deps.MODULE.bazel
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: build-windows-x86_64
- GitHub Check: build-linux-x86_64
174798d to
8b41326
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
third_party/rust/deps.MODULE.bazel (1)
7-17:⚠️ Potential issue | 🔴 CriticalFix required:
rustls-tlsfeature was renamed torustlsin reqwest 0.13.0Update line 11 to rename the feature from
"rustls-tls"to"rustls". This is a breaking change in reqwest 0.13.0 that causes the current build to fail with the error: "the package direct-cargo-bazel-deps depends on reqwest, with features: rustls-tls but reqwest does not have these features."Proposed fix
crate.spec( default_features = False, features = [ "blocking", - "rustls-tls", + "rustls", "http2", "charset", ], package = "reqwest", version = "=0.13.1", )
6da5561 to
0acd64e
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
third_party/rust/deps.MODULE.bazel (1)
7-17:⚠️ Potential issue | 🔴 CriticalBuild breakage:
rustls-tlsfeature was renamed torustlsin reqwest 0.13.x.The cargo feature
rustls-tlsdoes not exist in reqwest 0.13.0+; it was renamed torustls. Update line 11 to use the correct feature name.Proposed fix
crate.spec( default_features = False, features = [ "blocking", - "rustls-tls", + "rustls", "http2", "charset", ], package = "reqwest", version = "=0.13.2", )The other features (
blocking,http2,charset) are confirmed as valid in reqwest 0.13.x. However, note that in reqwest 0.13.0, thequeryandformfeatures were moved to separate opt-in crate features and are disabled by default. If your code relies on these, they must be explicitly added to the features list.
0acd64e to
0eadfee
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
third_party/rust/deps.MODULE.bazel (1)
7-17:⚠️ Potential issue | 🔴 CriticalFeature
rustls-tlswas renamed torustlsin reqwest v0.13.0 — build is broken.The
rustls-tlsfeature no longer exists in v0.13.2. It must be renamed torustls. The other features (blocking,http2,charset) remain valid in v0.13.2 and do not need changes.Fix: rename the feature
crate.spec( default_features = False, features = [ "blocking", - "rustls-tls", + "rustls", "http2", "charset", ], package = "reqwest", version = "=0.13.2", )
2382ce2 to
6636323
Compare
99033b0 to
85562c3
Compare
e2f79c8 to
5b28c34
Compare
This PR contains the following updates:
=0.12.28→=0.13.3=0.12.28→=0.13.3Release Notes
seanmonstar/reqwest (reqwest)
v0.13.3Compare Source
/etc/resolv.conffails.STOP_SENDINGas not an error.v0.13.2Compare Source
RequestBuilder::json()from override previously set content-type.v0.13.1Compare Source
v0.13.0Compare Source
rustlsis now the default TLS backend, instead ofnative-tls.rustlscrypto provider defaults to aws-lc instead of ring. (rustls-no-providerexists if you want a different crypto provider)rustls-tlshas been renamed torustls.rustls-platform-verifieris used by default.tls_certs_only(your_roots).native-tlsnow includes ALPN. To disable, usenative-tls-no-alpn.queryandformare now crate features, disabled by default.trust-dns, which was renamedhickory-dnsa while ago).tls_backend_rustls()overuse_rustls_tls().v0.12.28
v0.12.27
ClientBuilder::windows_named_pipe(name)option that will force all requests over that Windows Named Piper.v0.12.26
Accept-Encodingheader only with values configured with reqwest, regardless of underlying tower-http config.v0.12.25
Error::is_upgrade()to determine if the error was from an HTTP upgrade.Proxy-Authorizationif only username is configured.Proxy-Authorizationto HTTPS proxies when the target is HTTP.v0.12.24
v0.12.23
ClientBuilder::unix_socket(path)option that will force all requests over that Unix Domain Socket.ClientBuilder::retry(policy)andreqwest::retry::Builderto configure automatic retries.ClientBuilder::dns_resolver2()with more ergonomic argument bounds, allowing more resolver implementations.http3_*options toblocking::ClientBuilder.RequestBuilder.v0.12.22
v0.12.21
socks4a://instead ofsocks4h://.Error::is_timeout()to check for hyper and IO timeouts too.Errorto again include URLs when possible.DefaultforBody.v0.12.20
ClientBuilder::tcp_user_timeout(Duration)option to setTCP_USER_TIMEOUT.Error::is_status().v0.12.19
cookiesfeature is enabled (by the way, it's a noop feature in wasm).v0.12.18
socksenabled without TLS.v0.12.17
v0.12.16
ClientBuilder::http3_congestion_bbr()to enable BBR congestion control.ClientBuilder::http3_send_grease()to configure whether to send use QUIC grease.ClientBuilder::http3_max_field_section_size()to configure the maximum response headers.ClientBuilder::tcp_keepalive_interval()to configure TCP probe interval.ClientBuilder::tcp_keepalive_retries()to configure TCP probe count.Proxy::headers()to add extra headers that should be sent to a proxy.redirect::Policy::limit()which had an off-by-1 error, allowing 1 more redirect than specified.Response::bytes_stream().v0.12.15
ProxyOverrideandNO_PROXY.v0.12.14
fetch_mode_no_cors(), marking as deprecated when not on WASM.v0.12.13
Form::into_reader()for blockingmultipartforms.Form::into_stream()for asyncmultipartforms.RequestBuilder::form()from overwriting a previously setContent-Typeheader, like the other builder methods.blocking::Request.ProxyOverrideas aNO_PROXYvalue.Error::is_timeout()to return true when from a request timeout.v0.12.12
tokio/timeon WASM.v0.12.11
v0.12.10
ClientBuilder::connector_layer()to allow customizing the connector stack.ClientBuilder::http2_max_header_list_size()option.content-length) information when wrapping bodies.v0.12.9
tls::CertificateRevocationListssupport.connection_verbose()to output read logs.multipart::Part::file()to automatically include content-length.v0.12.8
multipart::Form::file()method for adding files easily.Body::wrap()to wrap anyhttp_body::Bodytype.v0.12.7
impl Service<http::Request<_>>forClient.v0.12.6
danger_accept_invalid_hostnamesforrustls.impl Service<http::Request<Body>>forClientand&'_ Client.!Syncbodies inBody::wrap_stream().hickory-dnsis used.Proxyso thatHTTP(S)_PROXYvalues take precedence overALL_PROXY.blocking::RequestBuilder::header()from unsettingsensitiveon passed header values.v0.12.5
blocking::ClientBuilder::dns_resolver()method to change DNS resolver in blocking client.http3feature back, still requiringreqwest_unstable.rustls-tls-no-providerCargo feature to use rustls without a crypto provider.Accept-Encodingheader combinations.v0.12.4
zstdsupport, enabled withzstdCargo feature.ClientBuilder::read_timeout(Duration), which applies the duration for each read operation. The timeout resets after a successful read.v0.12.3
FromStrfordns::Name.ClientBuilder::built_in_webpki_certs(bool)to enable them separately.ClientBuilder::built_in_native_certs(bool)to enable them separately.content-length: 0for GET requests.content_length()to return value when timeout is configured.ClientBuilder::resolve()to use lowercase domain names.v0.12.2
v0.12.1
ClientBuilder::interface()when no TLS is enabled.TlsInfo::peer_certificate()being truncated with rustls.http2feature disabled but TLS negotiated h2 in ALPN.DisplayforErrorto not include its source error.Configuration
📅 Schedule: (UTC)
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR has been generated by Mend Renovate.