[BUG] Cannot communicate with http2 when reactor-netty is enabled #18599
Conversation
|
❌ Gradle check result for 6cd5aef: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
| @@ -117,8 +159,6 @@ public Optional<TransportExceptionHandler> buildHttpServerExceptionHandler(Setti | |||
| @Override | |||
| public Optional<SSLEngine> buildSecureHttpServerEngine(Settings settings, HttpServerTransport transport) throws SSLException { | |||
| try { | |||
| var keyManagerFactory = KeyManagerFactory.getInstance("PKIX"); | |||
There was a problem hiding this comment.
Why remove these lines? I see now they are moved further up. ignore.
|
❌ Gradle check result for 6cd5aef: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
❌ Gradle check result for 6cd5aef: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #18599 +/- ##
============================================
+ Coverage 72.66% 72.78% +0.12%
- Complexity 68231 68329 +98
============================================
Files 5555 5556 +1
Lines 313857 313883 +26
Branches 45522 45527 +5
============================================
+ Hits 228073 228475 +402
+ Misses 67207 66810 -397
- Partials 18577 18598 +21 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Andriy Redko <[email protected]>
|
❌ Gradle check result for 6b27244: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
❌ Gradle check result for 6b27244: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Description
When an OpenSearch cluster is setup up http.type as reactor-netty-secure, the curl invocation of any endpoints fails. The reason is the client(curl) and the server try to communicate via http2. When http1.1 is forced we do not see the error, indicating the server fails to support http2 when reactor netty is enabled.
Related Issues
Closes #18559
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.