Merge pull request #43 from guardian/an/force-https-downloads

Forcibly upgrade all downloads to HTTPS
guardian · Aug 21, 2024 · 560b251 · 560b251
2 parents 65b1e66 + 2f01ee9
commit 560b251
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/associated-press/app/client/HttpClient.scala b/associated-press/app/client/HttpClient.scala
@@ -16,7 +16,11 @@ object HttpClient {
       headers: Seq[(String, String)] = Seq.empty,
       parameters: Seq[(String, String)] = Seq.empty
   ): Future[StandaloneWSResponse] = {
-    ws.url(uri)
+    // on the CODE env, the preview feed will return HTTP S3 URLs, but our
+    // security group only allows outbound requests on port 443...
+    // force all the URIs to HTTPS to allow the requests to complete.
+    val forcedHttps = uri.replaceAll("^http://", "https://")
+    ws.url(forcedHttps)
       .addHttpHeaders(headers: _*)
       .addQueryStringParameters(parameters: _*)
       .get()