-
Notifications
You must be signed in to change notification settings - Fork 49
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
Tweak docker base image, approval script, and sudoers file; add Docker section to README #124
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,6 +2,6 @@ Due to security and compliance requirements, this `sudo` session will require ap | |
|
||
To continue, another human must run: | ||
|
||
docker exec -it %h '%B %u %p' | ||
docker exec -it %h %B %u %p | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Having the command and its params in quotes causes Docker to look for a file of that exact name, which then can't be found. |
||
|
||
If a suitable human is not available and you have an immediate and urgent need to run this command, you may run the above command to approve your own session. [1mNote that doing so will immediately page an oncall security engineer, so this capability should only be used in the event of an emergency.[0m |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
nobody ALL = (: games) NOPASSWD: LOG_OUTPUT: ALL | ||
games ALL = (nobody) NOPASSWD:LOG_OUTPUT:ALL | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This might be wrong—I kind of reverse-engineered this from what seemed like the expected behavior for the environment; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sometime between the last update and now,
rust:latest
started building the plugin in such a way that it causes a segfault when added tosudo.conf
. Looking at bisecting that.