Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1599675 - Virt-who configuration script maintains passwords in simple text
Summary: Virt-who configuration script maintains passwords in simple text
Keywords:
Status: CLOSED DUPLICATE of bug 2158702
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Virt-who Configure Plugin
Version: 6.3.2
Hardware: x86_64
OS: Linux
low
low
Target Milestone: Unspecified
Assignee: Marek Hulan
QA Contact: Kunxin Huang
satellite-doc-list
URL:
Whiteboard:
: 2116590 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-07-10 11:10 UTC by Mihir Lele
Modified: 2023-05-01 13:16 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-02-03 16:30:09 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Mihir Lele 2018-07-10 11:10:26 UTC
Description of problem:

Virt-who configuration script that gets created after creating a virt-who profile in the Satellite webui maintains passwords in simple text

Version-Release number of selected component (if applicable):   6.3.2


How reproducible:  Always


Steps to Reproduce:
1. Go to sat webui, "Infrastructure" --> "virt-who Configurations" 
2. Create one virt-who profile
3. Go to sat webui, "Infrastructure" --> "virt-who Configurations"  --> Click on the existing profile and you will see all the passwords in clear text

Actual results:

Person having access to the sat webui also has the access to the hypervisor credentials due to this

Expected results:

Webui should not display passwords in clear text

Additional info:

Comment 1 Marek Hulan 2018-07-11 12:08:08 UTC
Password is encrypted in a database and decrypted when the configuration script is rendered. We need to store the password in a decryptable way so that we can transfer it to the machine where virt-who will be deployed. On that target machine, the password is then encrypted using virt-who's master password.

That's why we recommend using trusted channel while transfering the configuration script, best way is to use hammer, it uses https to access Satellite. But since some users don't install hammer on target machines, we also provide the configuration script in UI for "ctrl+c;ctrl+v" approach and there we need to display the password.

Is there a suggestion how to change to workflow?

Looking at the customer case, customer says "the issue is that the password is stored in plain text within the satellite server itself." which is not true. We just decrypt that on demand. So I'd suggest closing as not a bug.

Comment 3 Mihir Lele 2018-07-24 13:15:04 UTC
Hello,

The customer replied:

"ok it is stored encrypted. 

For us, displaying the Password in Plaintext is still an issue,  because the user for the hypervisor-access has broad credentials in the company. We do have a lot of satellite admins which not all should be able to know the password of that user. If, from your point of view, this is a "won't fix" problem, then you can close the call. We will continue to not use the gui-configuration for virt-who."


let me know if you still feel that this cannot be considered as an issue.

Comment 4 Marek Hulan 2018-07-24 15:33:54 UTC
Is there a recommendation from customer how to resolve this? We could drop the script (ctrl+c,ctrl+v) deployment mode and only support hammer from now on if that helps. Or we could made it hidden only but the password would be still in clipboard after user copies the value.

Comment 5 Mihir Lele 2018-08-07 13:49:12 UTC
Feedback from the customer:

could you add the option to directly feed the virt-who generated hash string itself to the sat6 webui instead of the plaintext password?
then the deployment script wouldn't require the plaintext password to generate the hash itself.

Comment 8 Marek Hulan 2019-08-23 13:42:59 UTC
> could you add the option to directly feed the virt-who generated hash string itself to the sat6 webui instead of the plaintext password?
then the deployment script wouldn't require the plaintext password to generate the hash itself.

We can't, virt-who uses master password which is always specific to host where you deploy it. We can't create the hash (or more precisely encrypt it) on Satellite side, since we don't know that master password. Note that we can't hash it, since virt-who needs to be able to decrypt the original value in order to be able to communicate with the compute resource.

Comment 15 Bryan Kearney 2020-01-15 21:00:57 UTC
The Satellite Team is attempting to provide an accurate backlog of bugzilla requests which we feel will be resolved in the next few releases. We do not believe this bugzilla will meet that criteria, and have plans to close it out in 1 month. This is not a reflection on the validity of the request, but a reflection of the many priorities for the product. If you have any concerns about this, feel free to contact Red Hat Technical Support or your account team. If we do not hear from you, we will close this bug out. Thank you.

Comment 16 Bryan Kearney 2020-02-03 16:30:09 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 17 Brad Buckingham 2022-08-11 14:41:15 UTC
*** Bug 2116590 has been marked as a duplicate of this bug. ***

Comment 18 Brad Buckingham 2023-05-01 13:16:54 UTC

*** This bug has been marked as a duplicate of bug 2158702 ***


Note You need to log in before you can comment on or make changes to this bug.