Bug #28628
closedAdd additional information to make oauth options description specific
Updated by Rahul Bajaj almost 5 years ago
- Assignee set to Rahul Bajaj
Description of problem:
As all of the option under hammer auth login oauth are related to oidc and protocol we are using for authentication is openidc, it is calid to change from oauth to oidc.
Version-Release number of selected component (if applicable):
Satellite 6.7 snap 7
tfm-rubygem-hammer_cli-0.19.0-2.el7sat.noarch
How reproducible:
always
Steps to Reproduce:
1. # hammer auth login oauth -h
Usage:
hammer auth login oauth [OPTIONS]
Options:
-a, --oidc-authorization-endpoint OPENIDC-AUTHORIZATION-ENDPOINT Openidc provider URL which issues authentication code
-c, --oidc-client-id OPENIDC-CLIENT-ID Client id used in the Openidc provider
-f, --two-factor Authenticate with two factor
-h, --help Print help
-p, --password PASSWORD Password to access the remote system
-r, --oidc-redirect-uri OPENIDC-REDIRECT-URI Redirect URI for the authencation code grant flow
-t, --oidc-token-endpoint OPENIDC-TOKEN-ENDPOINT Openidc provider URL which issues access token
-u, --username USERNAME Username to access the remote system
Actual results:
Command is hammer auth login oauth
Expected results:
Command should be hammer auth login oidc
Additional info:
Also there is typo, good to have it fixed
-r, --oidc-redirect-uri OPENIDC-REDIRECT-URI Redirect URI for the authencation code grant flow
s/authencation/authentication
Updated by The Foreman Bot almost 5 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/475 added
Updated by The Foreman Bot almost 5 years ago
- Fixed in Releases hammer-cli-foreman-0.20.0 added
Updated by Rahul Bajaj almost 5 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset hammer-cli-foreman|b4573406862c0abd92f101df3ccea51c9d5da683.
Updated by Rahul Bajaj almost 5 years ago
- Related to Tracker #28345: SSO using OpenID Connect added
Updated by Oleh Fedorenko almost 5 years ago
- Triaged changed from No to Yes
- Fixed in Releases hammer-cli-foreman-0.19.6 added