Skip to content

Commit 2e69f61

Browse files
authored
Merge pull request #133 from dracalafirex/main
fix(config): add support for sdk container credential provider
2 parents 4b0ff90 + daf802c commit 2e69f61

File tree

5 files changed

+69140
-68627
lines changed

5 files changed

+69140
-68627
lines changed

code-build.js

+10-4
Original file line numberDiff line numberDiff line change
@@ -280,10 +280,16 @@ function buildSdk() {
280280
customUserAgent: "aws-actions/aws-codebuild-run-build",
281281
});
282282

283-
assert(
284-
codeBuild.config.credentials && cloudWatchLogs.config.credentials,
285-
"No credentials. Try adding @aws-actions/configure-aws-credentials earlier in your job to set up AWS credentials."
286-
);
283+
// check if environment variable exists for the container credential provider
284+
if (
285+
!process.env.AWS_CONTAINER_CREDENTIALS_FULL_URI &&
286+
!process.env.AWS_CONTAINER_CREDENTIALS_RELATIVE_URI
287+
) {
288+
assert(
289+
codeBuild.config.credentials && cloudWatchLogs.config.credentials,
290+
"No credentials. Try adding @aws-actions/configure-aws-credentials earlier in your job to set up AWS credentials."
291+
);
292+
}
287293

288294
return { codeBuild, cloudWatchLogs };
289295
}

0 commit comments

Comments
 (0)