Skip to content

Support ConnectionSpec on Okhttp client and support null ConnectionSpec parameters #12677

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.

Sign up for GitHub

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

Draft
megla-tlanghorst wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Support ConnectionSpec on Okhttp client and support null ConnectionSpec parameters #12677

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3a79442
Allow proper customization of all SSL parameters
megla-tlanghorst Mar 5, 2026
d2729db
Make ConnectionSpec support null tlsVersions
megla-tlanghorst Mar 5, 2026
5fb58fe
Only check ALPN if TLS extensions are enabled
megla-tlanghorst Mar 6, 2026
37cb29c
Revert "Only check ALPN if TLS extensions are enabled"
megla-tlanghorst Mar 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Make ConnectionSpec support null tlsVersions
  • Loading branch information
@megla-tlanghorst
megla-tlanghorst committed Mar 5, 2026
commit d2729db12d2a62a92bae77f162df5d7dafe13df3
8 changes: 6 additions & 2 deletions okhttp/third_party/okhttp/main/java/io/grpc/okhttp/internal/ConnectionSpec.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -173,8 +173,12 @@ private ConnectionSpec supportedSpec(SSLSocket sslSocket, boolean isFallback) {
}
}

String[] protocolsToSelectFrom = sslSocket.getEnabledProtocols();
String[] protocolsToEnable = Util.intersect(String.class, tlsVersions, protocolsToSelectFrom);
String[] protocolsToEnable = null;
if (tlsVersions != null) {
String[] protocolsToSelectFrom = sslSocket.getEnabledProtocols();
protocolsToEnable = Util.intersect(String.class, tlsVersions, protocolsToSelectFrom);
}

return new Builder(this)
.cipherSuites(cipherSuitesToEnable)
.tlsVersions(protocolsToEnable)
Expand Down
Loading