Bug 679443 - Missing credentials shows misleading error message
Summary: Missing credentials shows misleading error message
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: user-experience
Version: unspecified
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: ---
Assignee: Pradeep Kilambi
QA Contact: Preethi Thomas
URL:
Whiteboard:
: 689554 (view as bug list)
Depends On:
Blocks: verified-to-close
TreeView+ depends on / blocked
 
Reported: 2011-02-22 15:33 UTC by Jay Dobies
Modified: 2013-09-09 16:29 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
: 700898 (view as bug list)
Environment:
Last Closed: 2011-08-16 12:08:28 UTC
Embargoed:


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.