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

AWS Credentials cache should be configurable and flexible instead of hard coded to 1 hour #36769

Open
aabchoo opened this issue Oct 22, 2024 · 3 comments
Labels

Comments

@aabchoo
Copy link

aabchoo commented Oct 22, 2024

Title: AWS Credentials cache should be configurable and flexible instead of hard coded to 1 hour

Description:

Describe the desired behavior, what scenario it enables and how it
would be used.

AWS access id, secret key, and session tokens read from AWS credential file are cached for 1 hour. This can result in stale credentials due to caching happening before session tokens are refreshed, or when an invalid token is cached.

The desired behavior is split into two parts:

  1. Allow users to configure the caching TTL time
  2. If the AWS credential file has been modified, clear the cache and read the keys/tokens from the updated file

Behaviour #1 allows us to shorten/extend the cache TTL to match the timeframe our tokens are valid for

Behaviour #2 allows us to update credential file adhoc and have those credentials be used by EnvoyProxy without needing to restart the application or wait for the cache TTL

[optional Relevant Links:]

Any extra documentation required to understand the issue.

Code where TTL is hardcoded

@aabchoo aabchoo added enhancement Feature requests. Not bugs or questions. triage Issue requires triage labels Oct 22, 2024
@mattklein123 mattklein123 added area/aws and removed enhancement Feature requests. Not bugs or questions. triage Issue requires triage labels Oct 22, 2024
@aabchoo
Copy link
Author

aabchoo commented Oct 23, 2024

The two ways that I've thought about implementing desired behavior number 2 is:

  • Process that watches for a signal/event when the credential file has been modified (issue arises when signal is missed)
  • Loop that will check every x seconds to check if the file has been updated (redundant checks)

In the event of a update, a flag fileUpdated will be set to true, and needsRefresh will use that value and evaluate to true (regardless of cache time)

Would appreciate opinions on this!

@mathetake
Copy link
Member

i think this request sounds reasonable

cc @suniltheta @nbaws

@nbaws
Copy link
Contributor

nbaws commented Oct 23, 2024

this seems reasonable. i will take a look at implementing something along these lines after i've finished curl deprecation patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants