evolix
/
ansible-roles
22
2
Fork 2
Code Issues 61 Pull Requests 18 Releases 32 Wiki Activity

packweb-apache: Use $SUDO_USER when possible in phpContainer #136

Closed
mtrossevin wants to merge 1 commits from mtrossevin/ansible-roles:use-sudo_user into unstable
pull from: mtrossevin/ansible-roles:use-sudo_user
merge into: evolix:unstable
Conversation 1 Commits 1 Files Changed 3 +5 -4
mtrossevin commented 2021-11-03 17:49:40 +01:00
Owner
Copy Link

And rollback to $LOGNAME when it isn't.

This remove the need to add $LOGNAME to env_keep in the sudoers file.

The $SUDO_USER variable is automatically set by sudo when it is used to
the user who used it so it fullfill the same role as $LOGNAME do in
phpContainer.

And rollback to $LOGNAME when it isn't. This remove the need to add $LOGNAME to env_keep in the sudoers file. The $SUDO_USER variable is automatically set by sudo when it is used to the user who used it so it fullfill the same role as $LOGNAME do in phpContainer.
mtrossevin added 1 commit 2021-11-03 17:49:43 +01:00
continuous-integration/drone/pr Build is failing Details
738f2682b0
packweb-apache: Use $SUDO_USER when possible in phpContainer 
And rollback to $LOGNAME when it isn't.

This remove the need to add $LOGNAME to env_keep in the sudoers file.

The $SUDO_USER variable is automatically set by sudo when it is used to
the user who used it so it fullfill the same role as $LOGNAME do in
phpContainer.
bwaegeneire commented 2021-11-15 11:55:27 +01:00
Owner
Copy Link

$LOGNAME is a more widely set varaible than $SUDO_USER, that's why it is used instead of the sudo one.

`$LOGNAME` is a more widely set varaible than `$SUDO_USER`, that's why it is used instead of the sudo one.
bwaegeneire closed this pull request 2021-11-15 11:55:27 +01:00
Some checks failed
continuous-integration/drone/pr Build is failing
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
security

Concern a security issue

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
invalid
question
security
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: evolix/ansible-roles#136
No description provided.
Delete Branch "mtrossevin/ansible-roles:use-sudo_user"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?