Skip to content

fix(action): forward AWS credentials into the build container - #57

Merged
XtremeOwnageDotCom merged 1 commit into
mainfrom
fix/action-forward-aws-credentials
Sep 2, 2026
Merged

fix(action): forward AWS credentials into the build container#57
XtremeOwnageDotCom merged 1 commit into
mainfrom
fix/action-forward-aws-credentials

Conversation

@XtremeOwnageDotCom

Copy link
Copy Markdown
Contributor

The action runs netdocs inside a container and passed through only HOME, so any plugin reaching AWS during the build had no credentials to resolve, however the surrounding workflow had configured them. An imported-docs S3 source failed with 'Failed to resolve AWS credentials' even directly after aws-actions/configure-aws-credentials succeeded.

Standard AWS_* variables present on the runner are now forwarded, and a web-identity token file is mounted and re-pointed at its in-container path.

What & why

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Refactor / chore

Checklist

  • dotnet format Netdocs.slnx --verify-no-changes passes
  • dotnet build Netdocs.slnx -c Release succeeds
  • dotnet test Netdocs.slnx -c Release passes
  • Added/updated tests for the change
  • Updated docs under docs-site/docs/** if behavior changed

Notes for reviewers

The action runs netdocs inside a container and passed through only HOME, so any plugin reaching AWS during the build had no credentials to resolve, however the surrounding workflow had configured them. An imported-docs S3 source failed with 'Failed to resolve AWS credentials' even directly after aws-actions/configure-aws-credentials succeeded.

Standard AWS_* variables present on the runner are now forwarded, and a web-identity token file is mounted and re-pointed at its in-container path.
@XtremeOwnageDotCom
XtremeOwnageDotCom merged commit 240634b into main Sep 2, 2026
1 check passed
@XtremeOwnageDotCom
XtremeOwnageDotCom deleted the fix/action-forward-aws-credentials branch September 2, 2026 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant