Bug 679443

Summary:	Missing credentials shows misleading error message
Product:	[Retired] Pulp	Reporter:	Jay Dobies <jason.dobies>
Component:	user-experience	Assignee:	Pradeep Kilambi <pkilambi>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	low	Docs Contact:
Priority:	low
Version:	unspecified	CC:	pkilambi, skarmark
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	700898 (view as bug list)		Environment:
Last Closed:	2011-08-16 12:08:28 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	647488

Description Jay Dobies 2011-02-22 15:33:38 UTC

Logged out to make sure I didn't have an admin certificate:

$ sudo pulp-admin auth logout
User credentials removed from [/home/jdob/.pulp]


I ran repo list without logging in or passing username/password. That error message is misleading, it should specify that the user is not logged in.

$ sudo pulp-admin repo list
error: operation failed: Invalid SSL Certificate

Comment 1 Preethi Thomas 2011-03-21 20:17:42 UTC

*** Bug 689554 has been marked as a duplicate of this bug. ***

Comment 2 Pradeep Kilambi 2011-04-13 19:02:53 UTC

commit 4cbeed60aa84fec995428274d7793a3c3888e47b

Added checks to:
 - see if user cert/key exists; 
 - if not; check if --username or --password is provided;
 - else throw a message 


$ sudo pulp-admin repo list
Error: No valid credentials; Please run `pulp-admin auth login` or use `--username` and `--password`

Comment 3 Jay Dobies 2011-04-13 19:23:00 UTC

Fixed in build 0.163.

Comment 4 Pradeep Kilambi 2011-04-14 15:35:42 UTC

Just note that with this fix, we exposed an issue in pulp-admin auth module:

Here is the bug: https://bugzilla.redhat.com/show_bug.cgi?id=696676

auth module should be using the top level credentials like every other command. Instead of asking for -u/-p again. Util this bug is fixed, you'll need to run auth login as

pulp-admin -u admin -p admin auth login --username admin --password admin

Comment 5 Preethi Thomas 2011-04-15 19:49:55 UTC

fails_qa

[root@preethi ~]# rpm -q pulp
pulp-0.0.163-1.fc14.noarch
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]#  pulp-admin auth logout
User credentials removed from [/root/.pulp]

[root@preethi ~]# pulp-admin repo list
error: operation failed: Invalid SSL Certificate

Comment 6 Pradeep Kilambi 2011-04-15 19:53:02 UTC

hrm are you sure you have the right version:

This is on my latest:


$ sudo pulp-admin auth logout
User credentials removed from [/root/.pulp]

$ sudo pulp-admin auth login
Error: No valid credentials; Please run `pulp-admin auth login` or use `--username` and `--password`

Comment 7 Preethi Thomas 2011-04-15 20:17:05 UTC

I have the correct version which is comment #3

[root@preethi ~]# rpm -qa pulp
pulp-0.0.163-1.fc14.noarch

Comment 8 Pradeep Kilambi 2011-04-15 20:21:55 UTC

looks like this fix hasnt made it into 163 build. Moving to Modified

Comment 9 Jeff Ortel 2011-04-19 18:49:16 UTC

build: 0.168

Comment 10 Preethi Thomas 2011-04-19 19:42:22 UTC

verified in 168

[root@preethi ~]# pulp-admin repo sync --id =f14
Error: No valid credentials; Please run `pulp-admin auth login` or use `--username` and `--password`

Comment 11 Preethi Thomas 2011-08-16 12:08:28 UTC

Closing with Community Release 15

pulp-0.0.223-4.

Comment 12 Preethi Thomas 2011-08-16 12:21:29 UTC

Closing with Community Release 15

pulp-0.0.223-4.