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

Allow ability to set max_session_duration for alks_iamtrustrole #43

Open
codezninja opened this issue Feb 6, 2019 · 4 comments
Open

Comments

@codezninja
Copy link
Contributor

codezninja commented Feb 6, 2019

I would like to manage the max_sessions_duration of a iam role through terraform. This is supported in the native aws provider. Is this something we can add?

@aaron-seitz
Copy link
Contributor

The Internal Tools team has discussed this some - given that the current default of 1 hr could be extended up to 12 hr, we think that it would be good for Security to take a peek at this to ensure there aren't any concerns there, especially in the context of L2 accounts.

We would appreciate some additional context behind the request and the use case it represents - what is the use case you're intending to use this for?

@jeremiahlukus
Copy link

@aaron-seitz any news on this?

I have a team that runs a script in jenkins to refresh our elasticsearch index. This process takes multiple hours to run. In order to do it we need the bento role attached. Increasing the time limit will allow for the script to finish before losing access to the prod account.

@jeremiahlukus
Copy link

@amagana3 ?

@codezninja
Copy link
Contributor Author

@aaron-seitz it's been a while. A few roles that get created through terraform might have sessions where they need a few hours to run. For example roles attached to ci/cd that run migrations that take a few hours.

Currently teams can workaround by manually updating the max session. This is less than ideal cause if we decided to recreate the role someone has to remember to manually update it.

What security concerns do you have if roles can be updated now manually with max_session_duration?

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

No branches or pull requests

3 participants