Project

General

Profile

Bug #23996

Hammer ask for password though -p option when user changing current password

Added by Ivan Necas 5 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Category:
Users and roles
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Team Backlog:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1580327

Description of problem: Though -p option passed to hammer, it will prompt user for password.

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

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.

Associated revisions

Revision 11ffa5de (diff)
Added by Aditi Puntambekar 3 months ago

Fixes #23996 - Hammer asks for user password though -p option provided

Revision cef2d066 (diff)
Added by Martin Bacovsky 3 months ago

Refs #23996 - Fix and extend tests for user update

History

#1 Updated by Tomáš Strachota 5 months ago

  • Subject changed from Hammer ask for password though -p option when user changing current password to Hammer ask for password though -p option when user changing current password

Btw it's possible to use the password from -p only when --login == the current user from -u or settings. It's not always the case. Admin users can change passwords of other users too.

#2 Updated by Tomáš Strachota 5 months ago

  • Category changed from Other commands to Users and roles

#3 Updated by Aditi Puntambekar 5 months ago

  • Triaged set to No
  • Assignee set to Aditi Puntambekar

#4 Updated by The Foreman Bot 5 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/371 added

#5 Updated by Aditi Puntambekar 3 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Martin Bacovsky 3 months ago

  • Fixed in Releases hammer-cli-foreman-0.13.2, hammer-cli-foreman-0.14.0 added

Also available in: Atom PDF