IamAwsProvider: fix url with role name

Fixes minio#1459 Signed-off-by: Bala.FA <[email protected]>
balamurugana · Dec 6, 2024 · 2512a93 · 2512a93
1 parent a7b443d
commit 2512a93
Showing 1 changed file with 6 additions and 10 deletions.
diff --git a/minio/credentials/providers.py b/minio/credentials/providers.py
@@ -505,21 +505,17 @@ def retrieve(self) -> Credentials:
             headers = {"X-aws-ec2-metadata-token": token} if token else None
 
             # Get role name
-            res = _urlopen(
-                self._http_client,
-                "GET",
-                urlunsplit(
-                    url_replace(
-                        urlsplit(url),
-                        path="/latest/meta-data/iam/security-credentials/",
-                    ),
+            url = urlunsplit(
+                url_replace(
+                    urlsplit(url),
+                    path="/latest/meta-data/iam/security-credentials/",
                 ),
-                headers=headers,
             )
+            res = _urlopen(self._http_client, "GET", url, headers=headers)
             role_names = res.data.decode("utf-8").split("\n")
             if not role_names:
                 raise ValueError(f"no IAM roles attached to EC2 service {url}")
-            url += "/" + role_names[0].strip("\r")
+            url += role_names[0].strip("\r")
         if not url:
             raise ValueError("url is empty; this should not happen")
         self._credentials = self.fetch(url, headers=headers)