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.

Sign up for GitHub

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

Jump to bottom

sudo commands giving error and allowed_shell_escape not working as expected #256

Open
Shashank-Pandey-ML opened this issue Jan 7, 2025 · 0 comments · May be fixed by #257
Open

sudo commands giving error and allowed_shell_escape not working as expected #256

Shashank-Pandey-ML opened this issue Jan 7, 2025 · 0 comments · May be fixed by #257

Comments

@Shashank-Pandey-ML
Copy link

I noticed 2 issues in the latest version of the lshell:

  1. Commands prefixed with sudo are giving this message:
admin:~$ sudo ls
sudo: a terminal is required to read the password; either use the -S option to read from standard input or configure an askpass helper
sudo: a password is required
  1. The allowed_shell_escape from the configuration is not working as expected.
Shashank-Pandey-ML added a commit to Shashank-Pandey-ML/lshell that referenced this issue Jan 7, 2025
@Shashank-Pandey-ML
Shell escape commands and sudo commands fix
c30feba 
- Fixed support for 'allowed_shell_escape' command.
- Allowing sudo commands to ask for password in the terminal.
Fixes ghantoos#256
@Shashank-Pandey-ML Shashank-Pandey-ML linked a pull request Jan 7, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Shell escape commands and sudo commands fix Shashank-Pandey-ML/lshell
1 participant
@Shashank-Pandey-ML