Bug 1580327

Summary: Hammer ask for password though -p option passed
Product: Red Hat Satellite Reporter: Nikhil Kathole <nkathole>
Component: HammerAssignee: Aditi Puntambekar <apuntamb>
Status: CLOSED ERRATA QA Contact: Jameer Pathan <jpathan>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.4CC: akarsale, apatel, apuntamb, aruzicka, dhlavacd, egolov, inecas, jpathan, kgaikwad, mhulan, nkathole, rabajaj, tstrachota
Target Milestone: 6.4.0Keywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: tfm-rubygem-hammer_cli_foreman-0.13.2-1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-10-16 19:10:02 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 Nikhil Kathole 2018-05-21 09:30:33 UTC 
Description of problem: Though -p option passed to hammer, it will prompt user for password.


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

# rpm -q satellite
satellite-6.4.0-7.beta.el7sat.noarch
# rpm -qa | grep hammer
tfm-rubygem-hammer_cli-0.13.0-1.el7sat.noarch


How reproducible: always


Steps to Reproduce:
1. # hammer -u admin -p <current_password> user update --login admin --password '<new-password>'
Enter user's current password: 
User [admin] updated.


Actual results: User prompted with 
Enter user's current password: 
though -p option is set.

Expected results: Should not ask again for user's current password if -p option is set.

Additional info:
Comment 3 Ivan Necas 2018-06-19 08:12:54 UTC 
Created redmine issue http://projects.theforeman.org/issues/23996 from this bug
Comment 9 Satellite Program 2018-08-08 18:06:03 UTC 
Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/23996 has been resolved.
Comment 10 Jameer Pathan 2018-08-24 11:26:24 UTC 
Verified:

# rpm -q satellite
satellite-6.4.0-11.beta.el7sat.noarch

#rpm -qa | grep hammer
tfm-rubygem-hammer_cli-0.13.1-1.el7sat.noarch

Step:
1. # hammer -u admin -p <current_password> user update --login admin --password '<new-password>'
User [admin] updated.
Comment 11 Bryan Kearney 2018-10-16 19:10:02 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2018:2927