Project

General

Profile

Bug #28628

Add additional information to make oauth options description specific

Added by Rahul Bajaj 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Team Backlog:
Found in Releases:
In Kanboard:

Related issues

Related to Foreman - Tracker #28345: SSO using OpenID ConnectNew

Associated revisions

Revision b4573406 (diff)
Added by Rahul Bajaj 5 months ago

Fixes #28628 - add specific oidc option information (#475)

History

#1 Updated by Rahul Bajaj 5 months 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

#2 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/475 added

#3 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases hammer-cli-foreman-0.20.0 added

#4 Updated by Rahul Bajaj 5 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by Rahul Bajaj 5 months ago

#6 Updated by Oleh Fedorenko 5 months ago

  • Triaged changed from No to Yes
  • Fixed in Releases hammer-cli-foreman-0.19.6 added

Also available in: Atom PDF