From 5814b4810059af57ecbf9c512131c325cdf909ad Mon Sep 17 00:00:00 2001 From: "Zach N." Date: Wed, 14 Feb 2024 07:28:44 -0800 Subject: [PATCH] fix(services/s3): environment/config role_arn ignored #4178 --- core/src/services/s3/backend.rs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/core/src/services/s3/backend.rs b/core/src/services/s3/backend.rs index bb7b3949c38..e07018513d3 100644 --- a/core/src/services/s3/backend.rs +++ b/core/src/services/s3/backend.rs @@ -897,6 +897,9 @@ impl Builder for S3Builder { if let Some(v) = self.config.security_token.take() { cfg.session_token = Some(v) } + if let Some(v) = self.config.role_arn.take() { + cfg.role_arn = Some(v) + } let mut loader: Option> = None; // If customed_credential_load is set, we will use it. @@ -905,7 +908,7 @@ impl Builder for S3Builder { } // If role_arn is set, we must use AssumeRoleLoad. - if let Some(role_arn) = self.config.role_arn.take() { + if let Some(role_arn) = cfg.role_arn.take() { // use current env as source credential loader. let default_loader = AwsDefaultLoader::new(client.client(), cfg.clone());