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

Least privilege #986

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

Least privilege #986

wants to merge 3 commits into from

Conversation

jonodrew
Copy link
Contributor

@jonodrew jonodrew commented Feb 6, 2025

I've rephrased the top line and reduced the privileged access session time.

@@ -32,7 +32,7 @@ Your team should:
- create the roles or credentials with the least possible privilege, with only necessary permissions required for normal users to perform their day-to-day jobs
- use the role or credentials with the least possible privilege as the default option
- use just-in-time (JIT) access provisioning to grant users an on-demand, time-limited privileged role or security token to access the privileged resources
- make sure session time of the privileged access is set to no more than 12 hours, and/or terminates when the user logs out of their laptop
- make sure session time of the privileged access is set to no more than 30 minutes, and/or terminates when the user logs out of their laptop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- make sure session time of the privileged access is set to no more than 30 minutes, and/or terminates when the user logs out of their laptop
- make sure session time of the privileged access (in non-development environments) is set to no more than 30 minutes, and/or terminates when the user logs out of their laptop

@galund
Copy link
Contributor

galund commented Feb 10, 2025

Noticed there's also a '12 hours' at line 47 that might want to be consistent

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.

2 participants