Disable protocol upgrades for the HTTP client by default.

This avoids problems where a connection could be blocked by an intermediate proxy that disallows upgrades. If wanted, it could be enabled on an individual case with a configuration API on the OTP HTTP client
opentripplanner · Oct 24, 2024 · 2e0a034 · 2e0a034
1 parent cee960f
commit 2e0a034
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/application/src/main/java/org/opentripplanner/framework/io/OtpHttpClient.java b/application/src/main/java/org/opentripplanner/framework/io/OtpHttpClient.java
@@ -378,6 +378,7 @@ private static RequestConfig requestConfig(Duration timeout) {
       .custom()
       .setResponseTimeout(Timeout.of(timeout))
       .setConnectionRequestTimeout(Timeout.of(timeout))
+      .setProtocolUpgradeEnabled(false)
       .build();
   }
 

diff --git a/application/src/main/java/org/opentripplanner/framework/io/OtpHttpClientFactory.java b/application/src/main/java/org/opentripplanner/framework/io/OtpHttpClientFactory.java
@@ -132,6 +132,7 @@ private static RequestConfig requestConfig(Duration timeout) {
       .custom()
       .setResponseTimeout(Timeout.of(timeout))
       .setConnectionRequestTimeout(Timeout.of(timeout))
+      .setProtocolUpgradeEnabled(false)
       .build();
   }
 }