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

remove root form "sudo -H -u root bash -c" #731

Closed

jacktd9 opened this issue Aug 30, 2020 · 1 comment

Labels

status/TODO type/feature-request

Milestone

jacktd9 commented Aug 30, 2020

tiup 运维使用中有写定 sudo root 一类指令，
在用户实际环境下含 root 的调用指令存在安全规范问题，如果 tiup 写定了root 就用不了。

了解应是可以去除 root 吧
sudo bash -c xxx 等价于 sudo -u root bash xxx

jacktd9 added the type/feature-request label

jacktd9 changed the title ~~from~~ remove root form "sudo -H -u root bash -c"

lucklove added the status/TODO label

lucklove added this to the v1.2.0 milestone

lucklove added a commit to lucklove/tiup that referenced this issue


          Remove username from sudo command

e8e2226

Fix pingcap#731

Signed-off-by: lucklove <[email protected]>

Member

lucklove commented Sep 1, 2020

Fixed by #738

lucklove closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment