Bug 1771367

Summary:	undefined method `request_uri' when Openidc Provider Token Endpoint is none
Product:	Red Hat Satellite	Reporter:	Nikhil Kathole <nkathole>
Component:	Authentication	Assignee:	Rahul Bajaj <rabajaj>
Status:	CLOSED ERRATA	QA Contact:	Omkar Khatavkar <okhatavk>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.7.0	CC:	apatel, egolov, kgaikwad, mhulan, pcreech, tbrisker
Target Milestone:	6.8.0	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	tfm-rubygem-hammer_cli_foreman-0.20.0, foreman-2.1.0-0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-10-27 12:59:24 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Nikhil Kathole 2019-11-12 09:14:26 UTC

Description of problem:
undefined method `request_uri' when Openidc Provider Token Endpoint is none


Version-Release number of selected component (if applicable):
Satellite 6.7 snap 1


How reproducible: always


Steps to Reproduce:
1. set :use_sessions: true in .hammer/cli.modules.d/foreman.yml
2. hammer auth login oauth
Openidc Provider Token Endpoint: 
Client ID: 
Username: 
Password: 
Error: undefined method `request_uri' for #<URI::Generic >

Actual results:
Error: undefined method `request_uri' for #<URI::Generic >

Expected results:
Error should be handled and throw valid message.


Additional info:

Comment 5 Bryan Kearney 2019-11-13 15:04:09 UTC

Upstream bug assigned to rabajaj

Comment 6 Rahul Bajaj 2019-11-20 18:52:49 UTC

Hello,

Please find the PR for this issue here: https://github.com/theforeman/hammer-cli-foreman/pull/453

Thanks,

Comment 7 Bryan Kearney 2019-11-21 13:04:40 UTC

Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/28196 has been resolved.

Comment 8 Nikhil Kathole 2019-12-02 14:18:31 UTC

FailedQA

Version tested:
Satellite 6.7 snap 4


# hammer auth login oauth 
Openidc Provider Token Endpoint: https://
Client ID: apv
Username: d
Password: 
Error: undefined method `ascii_only?' for nil:NilClass

# hammer auth login oauth 
Openidc Provider Token Endpoint: fsf
Client ID: fds
Username: fds
Password: 
Error: undefined method `request_uri' for #<URI::Generic fsf>

Comment 9 Rahul Bajaj 2020-01-17 08:49:48 UTC

Hello,

The issue has been fixed and is a part of the hammer-cli-foreman 0.19.6. Therefore, adding this one to ON_QA.

Thanks,

Comment 10 Rahul Bajaj 2020-01-21 10:31:41 UTC

Hello,

Sorry, the corresponding Foreman PR is still in review: https://github.com/theforeman/foreman/pull/7335
Moving this to ON_DEV.

Thanks,

Comment 20 errata-xmlrpc 2020-10-27 12:59:24 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Important: Satellite 6.8 release), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:4366