Allow admins to sudo to blocked users
What does this MR do?
- Allows admin users to
sudoto blocked users - This presents one way to solve #29505 (closed)
Are there points in the code the reviewer needs to double check?
- Are there any unintended consequences of this change I didn't think of?
- Is there a reason this limitation was deliberately introduced in the first place?
Why was this MR needed?
Admins currently cannot delete snippets for blocked users. We could solve this by modifying the snippets API to allow admin access, but it is a lot easier to change this at the sudo level.
What are the relevant issue numbers?
- Related to #29505 (closed)
Notes
- We don't need to disable updating user login metadata (
current_sign_in_at,last_sign_in_at, etc.) when usingsudowith a blocked user, since API calls already don't update a user's login metadata. API authentication simply setscurrent_userwithout calling out towarden.set_useror Devise'ssign_in, so the hook forTrackableis not run, and the metadata is not updated.
Edited by 🤖 GitLab Bot 🤖