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

-i and -s flag are not supported properly by the CLI #225

Closed
squell opened this issue Apr 17, 2023 · 0 comments · Fixed by #226
Closed

-i and -s flag are not supported properly by the CLI #225

squell opened this issue Apr 17, 2023 · 0 comments · Fixed by #226
Assignees
Labels
C-cli CLI interface
Milestone

Comments

@squell
Copy link
Member

squell commented Apr 17, 2023

sudo -i and sudo -s should also run without arguments:

$ sudo-rs -i
sudo-rs: `': command not found

In original sudo, this gives you a shell into the root home directory. Similarly sudo -s is of course what everybody should use instead of sudo bash or sudo su.

Note that this should not be dealt with in the sudo-exec crate, since that runs post-permission checks (we must check that the user actually has permission to execute the shell)

@squell squell added the C-cli CLI interface label Apr 17, 2023
@squell squell mentioned this issue Apr 17, 2023
@squell squell self-assigned this Apr 17, 2023
@squell squell linked a pull request Apr 17, 2023 that will close this issue
@squell squell added this to the Milestone 2 milestone Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cli CLI interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant