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

CLI Overview #129

Open
pvdrz opened this issue Mar 21, 2023 · 0 comments
Open

CLI Overview #129

pvdrz opened this issue Mar 21, 2023 · 0 comments
Labels
C-cli CLI interface

Comments

@pvdrz
Copy link
Collaborator

pvdrz commented Mar 21, 2023

Since we decided we don't need to be a perfect, in-place replacement for sudo, we should consider removing some of the CLI options that we added. I'm opening this issue to keep track of this discussion.

Already implemented

Planned

On the wishlist, but not scheduled

To be decided

  • --preserve-env=list
  • -R directory, --chroot=directory (similar to chdir)
  • -P, --preserve-groups
  • -b, --background

Doubtful

  • -B, --bell
  • -C num, --close-from=num
  • -T timeout, --command-timeout=timeout
  • -p prompt, --prompt=prompt
  • -H, --set-home

Highly doubtful

  • -A, --askpass
  • -h host, --host=host
@squell squell mentioned this issue Mar 28, 2023
@squell squell changed the title CLI clean up CLI Overview Apr 18, 2023
Mynacol added a commit to Mynacol/nixpkgs that referenced this issue Jul 2, 2024
This helps supporting sudo-rs, which currently does not implement the
--preserve-env flag and probably won't so in the foreseeable future [1].

The replacement just sets both environment variables behind the sudo
invocation with env, as sudo-rs also doesn't implement env var lists.

The OC_PASS variable is dropped, as it is seemingly unused and would
leak through this approach through /proc.

[1] trifectatechfoundation/sudo-rs#129
@squell squell added the C-cli CLI interface label Aug 26, 2024
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

No branches or pull requests

2 participants