Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CHEF-85 auth-token-refresh-retry-for-inspec-aws #970

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

sa-progress
Copy link
Contributor

@sa-progress sa-progress commented Apr 25, 2023

Description

Issue: AWS session expiration after 12 hours using AssumeRole credentials

Currently, when using AWS resources, the session expires after 12 hours. This can cause disruptions to long-running scans that rely on these credentials.

Fix: To address this issue, the recommended approach is to implement a scheduled rotation of the AssumeRole credentials before the expiration time. This can be achieved by using AssumeRole API and retrieve a new set of temporary credentials before the existing ones expire.

Additional env variables required :
export AWS_ROLE_ARN="arn:aws:iam::1127dummy395563:role/DUMMYRole"
export AWS_TOKEN_EXPIRATION_DURATION="901"
export AWS_ROLE_SESSION_NAME="DUMMY_aws_role_for_session"

Issues Resolved

List any existing issues this PR resolves, or any Discourse or StackOverflow discussion that's relevant

Check List

Please fill box or appropriate ([x]) or mark N/A.

@sa-progress sa-progress requested a review from a team as a code owner April 25, 2023 13:01
@sa-progress sa-progress self-assigned this Apr 25, 2023
@netlify
Copy link

netlify bot commented Apr 25, 2023

Deploy Preview for inspec-aws canceled.

Name Link
🔨 Latest commit b3d4c40
🔍 Latest deploy log https://app.netlify.com/sites/inspec-aws/deploys/648878c359102f000810e656

@sa-progress sa-progress added Version: Bump Minor Used by github.minor_bump_labels to bump the Minor version number. and removed Do not merge labels May 9, 2023
Signed-off-by: sa-progress <[email protected]>
Signed-off-by: sa-progress <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Version: Bump Minor Used by github.minor_bump_labels to bump the Minor version number.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants