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

Prefix sudoers lines with the user that is logging in instead of requiring a trait be templated. #14007

Merged
merged 1 commit into from
Jul 1, 2022

Conversation

lxea
Copy link
Contributor

@lxea lxea commented Jun 30, 2022

Instead of doing

{{internal.logins}} ALL=(ALL) ALL

this line can be

ALL=(ALL) ALL

and this line will be prefixed the user that is logging in.

@lxea lxea force-pushed the lxea/sudoers-name-prefix branch from 48b5955 to 3fa326c Compare June 30, 2022 14:46
Copy link
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me.

lib/srv/usermgmt.go Outdated Show resolved Hide resolved
@lxea lxea enabled auto-merge (squash) July 1, 2022 08:37
@lxea lxea force-pushed the lxea/sudoers-name-prefix branch from 3fa326c to e8bce57 Compare July 1, 2022 08:37
@lxea lxea merged commit aee44e5 into master Jul 1, 2022
@github-actions
Copy link

github-actions bot commented Jul 1, 2022

@lxea See the table below for backport results.

Branch Result
branch/v10 Create PR

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.

4 participants