Skip to content

Commit 3b3539a

Browse files
authored
add http read timeout to IMDS requests (fixes #267) (#287)
1 parent 5a3300c commit 3b3539a

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

CHANGES.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
# aws-api
22

3+
## DEV
4+
5+
* Add http read timeout to IMDS requests [#267](https://github.com/cognitect-labs/aws-api/issues/267)
6+
37
## 0.8.762 / 2025-08-05
48

59
* Add Babashka support

src/cognitect/aws/ec2_metadata_utils.clj

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,10 @@
4747

4848
(defn- request-map
4949
[^URI uri]
50-
(let [auth-token (u/getenv container-authorization-token-env-var)]
50+
(let [auth-token (u/getenv container-authorization-token-env-var)
51+
;; matches the java sdk v2 default value
52+
;; https://github.com/aws/aws-sdk-java-v2/blob/43950cfe9c067b56f3eedaa8c078432495be7c36/core/sdk-core/src/main/java/software/amazon/awssdk/core/SdkSystemSetting.java#L93-L101
53+
read-timeout-msec 1000]
5154
{:scheme (.getScheme uri)
5255
:server-name (.getHost uri)
5356
:server-port (or (when (pos? (.getPort uri)) (.getPort uri))
@@ -57,7 +60,9 @@
5760
:request-method :get
5861
:headers (cond-> {"Accept" "*/*"}
5962
auth-token
60-
(assoc "Authorization" auth-token))}))
63+
(assoc "Authorization" auth-token))
64+
:timeout-msec read-timeout-msec
65+
:cognitect.http-client/timeout-msec read-timeout-msec}))
6166

6267
(defn- get-response-data [request-map http-client]
6368
(let [response (a/<!! (retry/with-retry

0 commit comments

Comments
 (0)