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

sudo chef-server-ctl user-create shouldn't show the key in the terminal #1808

Open
tas50 opened this issue Oct 25, 2019 · 1 comment
Open
Labels
Aspect: Security Can an unwanted third party affect the stability or look at privileged information? Component: chef-server-ctl Status: Good First Issue An issue ready for a new contributor. Status: To be prioritized Indicates that product needs to prioritize this issue. Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected.

Comments

@tas50
Copy link
Contributor

tas50 commented Oct 25, 2019

Chef Server Version

13.0

Platform Details

Ubuntu 18.04

Configuration

Standalone

Scenario:

When the user runs 'sudo chef-server-ctl user-create ...' it currently creates the private key in the terminal. This means the private key is available in terminal scrollback which can be infinite for a lot of users. This seems like a bad security setup. We should just write the key to disk and then tell the user where to find it. That's how a lot of similar CLI tools handle key generation so they don't save secrets into insecure mediums.

@tas50 tas50 added the Status: Untriaged An issue that has yet to be triaged. label Oct 25, 2019
@PrajaktaPurohit PrajaktaPurohit added Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected. Aspect: Security Can an unwanted third party affect the stability or look at privileged information? Component: chef-server-ctl Size: S Status: Good First Issue An issue ready for a new contributor. Status: To be prioritized Indicates that product needs to prioritize this issue. and removed Status: Untriaged An issue that has yet to be triaged. labels Nov 1, 2019
@kaorihinata
Copy link
Contributor

Probably worth mentioning that chef-server-ctl actually wraps knife-opc to provide most of its organizational management functionality. It also wraps it pretty closely (see: wrap-knife-opc.rb), so there's not really much room to override this functionality in chef-server-ctl. You may want to move this issue over to knife-opc.

@tas50 tas50 removed the Size: S label May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aspect: Security Can an unwanted third party affect the stability or look at privileged information? Component: chef-server-ctl Status: Good First Issue An issue ready for a new contributor. Status: To be prioritized Indicates that product needs to prioritize this issue. Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected.
Projects
None yet
Development

No branches or pull requests

3 participants