Enable HTTP/2 via default Reqwest transport and log negotiated protocol#195
Merged
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
Contributor
Author
|
@pubnub-release-bot release |
pubnub-release-bot
requested review from
kazydek and
techwritermat
as code owners
|
🚀 Release successfully completed 🚀 |
AkiraMiyakoda
pushed a commit
to AkiraMiyakoda/pubnub-rust
that referenced
this pull request
Jun 14, 2026
…ol (pubnub#195) feat: Enable HTTP/2 negotiation on the default TransportReqwest by turning on reqwest’s native-tls-alpn feature, so ALPN can select h2 when the origin supports it while preserving HTTP/1.1 fallback. refactor: Add debug-level protocol logging in async and blocking TransportReqwest::send() after each completed request, including HTTP method, path, and the Reqwest-reported protocol version. fix: Fix a Clippy derivable_impls lint in RequestRetryConfiguration (derive Default on the enum instead of a manual impl).
AkiraMiyakoda
pushed a commit
to AkiraMiyakoda/pubnub-rust
that referenced
this pull request
Jun 14, 2026
…ol (pubnub#195) feat: Enable HTTP/2 negotiation on the default TransportReqwest by turning on reqwest’s native-tls-alpn feature, so ALPN can select h2 when the origin supports it while preserving HTTP/1.1 fallback. refactor: Add debug-level protocol logging in async and blocking TransportReqwest::send() after each completed request, including HTTP method, path, and the Reqwest-reported protocol version. fix: Fix a Clippy derivable_impls lint in RequestRetryConfiguration (derive Default on the enum instead of a manual impl).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
feat: Enable HTTP/2 negotiation on the default
TransportReqwestby turning on reqwest’snative-tls-alpnfeature, so ALPN can selecth2when the origin supports it while preserving HTTP/1.1 fallback.refactor: Add debug-level protocol logging in async and blocking
TransportReqwest::send()after each completed request, including HTTP method, path, and the Reqwest-reported protocol version.fix: Fix a Clippy
derivable_implslint inRequestRetryConfiguration(deriveDefaulton the enum instead of a manual impl).