Bug 1659068

Summary: [RFE] - Password should not be saved in plain text for hammer
Product: Red Hat Satellite Reporter: Anand Jambhulkar <ajambhul>
Component: HammerAssignee: satellite6-bugs <satellite6-bugs>
Status: CLOSED WONTFIX QA Contact: Suraj Bora <sbora>
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.4.0CC: akarsale, apatel, bkearney, dhlavacd, kgaikwad, mbacovsk, mhulan, mshira, rabajaj, tstrachota
Target Milestone: UnspecifiedKeywords: FutureFeature
Target Release: Unused   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-12-13 18:08:40 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Anand Jambhulkar 2018-12-13 14:04:12 UTC
1. What is the exact nature of the problem trying to be solved with this request?
Satellite admin password is saved in plaintext in " /root/.hammer/cli.modules.d/foreman.yml" file for hammer cli command execution. We would like the file to save encrypted password and not in plaintext.

    2. List the business requirements.
Password should not be saved in plaintext on any server.

    3. Do you have specific timeline dependencies?
We have professional services onsite helping us with satellite deployment so would like know if there is a way we can force hammer to save password in encypted format. If not please add this as a new feature in next satellite version.

    4. Can you please explain us the what are the functional requirements? If test cases can be provided this would be even more ideal.
Password should not be saved in plaintext format.

    5. Would you be able to assist in testing this functionality if implemented?
I believe Red Hat should be able to test and verify this functionality on satellite application.

Comment 6 Bryan Kearney 2018-12-13 18:08:40 UTC
Thank you for your interest in Satellite 6. We have evaluated this request, and while we recognize that it is a valid request, we do not expect this to be implemented in the product in the foreseeable future. This is due to other priorities for the product, and not a reflection on the request itself. We are therefore closing this out as WONTFIX. If you have any concerns about this, please do not reopen. Instead, feel free to contact Red Hat Technical Support. Thank you.

Comment 7 Marek Hulan 2018-12-17 15:16:20 UTC
To avoid storing passwords in plaintext, one can use personal access tokens. There's API and hammer support for creating the token, that can be then used instead of a password. Also note that as long as the file is owner by root and set to 0600, it has the same level of security as the virt-who-password tool.