gravitational / teleport

The easiest, and most secure way to access and protect all of your infrastructure.
https://goteleport.com
GNU Affero General Public License v3.0
17.37k stars 1.74k forks source link

Teleport should auto-lock accounts during common actions (for example account delete) #40098

Open jentfoo opened 5 months ago

jentfoo commented 5 months ago

Whenever an account or session must be immediately denied, our locking functionality must be used: https://goteleport.com/docs/access-controls/guides/locking/

This is not always intuitive, and customers may be unaware that this is required when an account is deleted or roles are changed. We should consider auto-adding locks where it is reasonable and easy to do so. For example:

Raised in part from this issue: https://github.com/gravitational/security-findings/issues/73

zmb3 commented 5 months ago

This has come up several times before for access requests, and we have decided not to do it.

Whatever we do, it should probably be consistent. I would not expect this behavior to differ for users and access requests.

jentfoo commented 5 months ago

This seems to be a frequent source of confusion, so I think we should continue to consider this