FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(gax): register Conscrypt SSLContext SPI classes for GraalVM reflection by lqiu96 · Pull Request #14129 · googleapis/google-cloud-java · GitHub

fix(gax): register Conscrypt SSLContext SPI classes for GraalVM reflection - #14129

Merged
lqiu96 merged 9 commits into
googleapis:mainfrom
lqiu96:fix_sponge_failure_autorepair
Aug 20, 2026
Merged

fix(gax): register Conscrypt SSLContext SPI classes for GraalVM reflection#14129
lqiu96 merged 9 commits into
googleapis:mainfrom
lqiu96:fix_sponge_failure_autorepair

Conversation

lqiu96 commented Aug 19, 2026

Copy link
Copy Markdown
Member

Error:

  JUnit Jupiter:ITBigQueryStorageTest
    ClassSource [className = 'com.google.cloud.bigquery.storage.v1beta2.it.ITBigQueryStorageTest', filePosition = null]
    => java.lang.IllegalStateException: Failed to initialize SSLSocketFactory.
       com.google.api.client.http.javanet.NetHttpTransport$Builder.createDefaultSslSocketFactory(NetHttpTransport.java:476)
       com.google.api.client.http.javanet.NetHttpTransport$Builder.resolveSslSocketFactory(NetHttpTransport.java:446)
       com.google.api.client.http.javanet.NetHttpTransport$Builder.build(NetHttpTransport.java:485)
       com.google.cloud.http.HttpTransportOptions$DefaultHttpTransportFactory.create(HttpTransportOptions.java:75)
       com.google.cloud.bigquery.spi.v2.HttpBigQueryRpc.<init>(HttpBigQueryRpc.java:108)
       [...]
     Caused by: java.security.NoSuchAlgorithmException: class configured for SSLContext (provider: Conscrypt) cannot be found.
       java.base@21.0.2/java.security.Provider$Service.getImplClass(Provider.java:2004)
       java.base@21.0.2/java.security.Provider$Service.getDefaultConstructor(Provider.java:2020)
       java.base@21.0.2/java.security.Provider$Service.newInstanceOf(Provider.java:1934)
       java.base@21.0.2/java.security.Provider$Service.newInstanceUtil(Provider.java:1942)
       java.base@21.0.2/java.security.Provider$Service.newInstance(Provider.java:1917)
       [...]
     Caused by: java.lang.ClassNotFoundException: org.conscrypt.OpenSSLContextImpl$TLSv13
       java.base@21.0.2/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:52)
       java.base@21.0.2/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
       java.base@21.0.2/java.lang.ClassLoader.loadClass(ClassLoader.java:121)
       java.base@21.0.2/java.security.Provider$Service.getImplClass(Provider.java:1991)
       [...]
       ```

lqiu96 requested a review from whowes August 19, 2026 20:54
lqiu96 requested review from a team as code owners August 19, 2026 20:54

gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Code Review

This pull request adds GraalVM reflection registration support for Conscrypt SSLContext and Security Provider SPI implementation classes when Conscrypt is present on the classpath. The reviewer noted that nested classes such as KeyManagerFactoryImpl$X509 and TrustManagerFactoryImpl$X509 are not automatically registered by registerClassHierarchyForReflection and will cause ClassNotFoundException at runtime. They also suggested registering older TLS versions (TLSv11 and TLSv1) to support older protocol negotiations.

lqiu96 force-pushed the fix_sponge_failure_autorepair branch from 043c8ea to ba87cda Compare August 19, 2026 21:06
lqiu96 enabled auto-merge (squash) August 19, 2026 21:15
lqiu96 disabled auto-merge August 19, 2026 22:03
lqiu96 merged commit 73c0243 into googleapis:main Aug 20, 2026
285 of 289 checks passed
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL