Actions
Bug #20315
closedStored credentials should be mutually exclusive with auth sessions
Status:
Closed
Priority:
Normal
Assignee:
Category:
Other commands
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1471099
Description of problem:
If .hammer/cli.modules.d/foreman.yml has username and password specified as well as use_sessions enabled, the stored credentials will take precedence over the session.
If you specify ":use_sessions: true" and don't erase username/password, you'll see:
~]# hammer -u admin2 -p changeme organization list ... ~]# hammer auth status Session exist, currently logged in as 'admin2' ~]# hammer organization list ... ~]# hammer auth status Session exist, currently logged in as 'admin'
So after running "hammer organization list" the session is reset to what is specified in the file.
Also if you specify just username in the file, you are prompted for password on a command run, but it creates problems when attempting to change session to a different user:
~]# hammer -u admin2 -p changeme organization list ... ~]# hammer auth status Session exist, currently logged in as 'admin2' ~]# hammer organization list [Foreman] Password for admin:
Updated by Tomáš Strachota over 7 years ago
- Subject changed from Stored credentials should be mutually exclusive with auth sessions to Stored credentials should be mutually exclusive with auth sessions
- Status changed from New to Assigned
- Target version set to 1.14.3
Updated by The Foreman Bot over 7 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/320 added
Updated by Marek Hulán over 7 years ago
- Target version changed from 1.14.3 to 1.17.0-RC2
Updated by Marek Hulán over 7 years ago
- Related to Bug #20131: Unsuccessful `hammer auth login` takes down existing session added
Updated by Tomáš Strachota over 7 years ago
- Translation missing: en.field_release set to 243
Updated by Anonymous over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset hammer-cli-foreman|0de9a76f32087f03681d6a50ae8b42c8f3a729a2.
Updated by The Foreman Bot over 7 years ago
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/322 added
Actions