Project

General

Profile

Actions

Bug #28628

closed

Add additional information to make oauth options description specific

Added by Rahul Bajaj over 4 years ago. Updated over 4 years ago.

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

Related issues 1 (1 open0 closed)

Related to Foreman - Tracker #28345: SSO using OpenID ConnectNewRahul Bajaj

Actions
Actions #1

Updated by Rahul Bajaj over 4 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

Actions #2

Updated by The Foreman Bot over 4 years ago

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

Updated by The Foreman Bot over 4 years ago

  • Fixed in Releases hammer-cli-foreman-0.20.0 added
Actions #4

Updated by Rahul Bajaj over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions #5

Updated by Rahul Bajaj over 4 years ago

Actions #6

Updated by Oleh Fedorenko over 4 years ago

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

Also available in: Atom PDF