Bug 1148919

Summary:	Traceback is shown in consumer.log if pulp-consumer experiences an authentication failure
Product:	[Retired] Pulp	Reporter:	Irina Gulina <igulina>
Component:	consumers	Assignee:	amacdona <austin>
Status:	CLOSED UPSTREAM	QA Contact:	Irina Gulina <igulina>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	2.5	CC:	rbarlow
Target Milestone:	---
Target Release:	2.6.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-28 22:37:34 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 Irina Gulina 2014-10-02 16:08:39 UTC

Description of problem:
If pulp-consumer is used without proper authentication, a traceback is logged on the .pulp/consumer.log. Logging a failed authentication is perfectly reasonable, but logging a traceback is not. Remove the traceback and instead log a simple message about the failed authentication attempt, including the username attempted.

Version-Release number of selected component (if applicable):
>> rpm -qa | grep pulp-server
pulp-server-2.5.0-0.6.beta.fc20.noarch

How reproducible:
always

Steps to Reproduce:
>> pulp-consumer -u admin -p wrong_pass register --consumer-id gena
Authentication Failed
 
A valid Pulp user is required to register a new consumer. Please double check
the username and password and attempt the request again.
 
>> less .pulp/consumer.log
 
2014-10-02 14:08:49,389 - ERROR - Client-side exception occurred
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/pulp/client/extensions/core.py", line 478, in run
    exit_code = Cli.run(self, args)
  File "/usr/lib/python2.7/site-packages/okaara/cli.py", line 974, in run
    exit_code = command_or_section.execute(self.prompt, remaining_args)
  File "/usr/lib/python2.7/site-packages/pulp/client/extensions/extensions.py", line 224, in execute
    return self.method(*arg_list, **clean_kwargs)
  File "/usr/lib/python2.7/site-packages/pulp/client/consumer/cli.py", line 190, in register
    rsa_pub=rsa_pub)
  File "/usr/lib/python2.7/site-packages/pulp/bindings/consumer.py", line 47, in register
    return self.server.POST(path, body)
  File "/usr/lib/python2.7/site-packages/pulp/bindings/server.py", line 99, in POST
    return self._request('POST', path, body=body, ensure_encoding=ensure_encoding)
  File "/usr/lib/python2.7/site-packages/pulp/bindings/server.py", line 151, in _request
    self._handle_exceptions(response_code, response_body)
  File "/usr/lib/python2.7/site-packages/pulp/bindings/server.py", line 192, in _handle_exceptions
    raise code_class_mappings[response_code](response_body)
PermissionsException: RequestException: POST request on /pulp/api/v2/consumers/ failed with 401 - Authentication with username admin failed: invalid username or password


Expected results:
PermissionsException: RequestException: POST request on /pulp/api/v2/consumers/ failed with 401 - Authentication with username admin failed: invalid username or password

Comment 1 Randy Barlow 2014-10-02 18:14:33 UTC

Austin, this is low hanging fruit if you want an easy BZ.

Comment 2 amacdona@redhat.com 2014-10-06 18:52:58 UTC

Fixed by https://github.com/pulp/pulp/pull/1190

Comment 3 Chris Duryee 2014-12-23 20:52:21 UTC

fixed in pulp 2.6.0-0.2.beta

Comment 4 Irina Gulina 2014-12-24 00:20:56 UTC

>> rpm -qa | grep pulp-server
pulp-server-2.6.0-0.2.beta.fc20.noarch

>> pulp-consumer -u admin -p wrong_pass register --consumer-id gena
Authentication with username admin failed: invalid username or password

>> less .pulp/consumer.log

2014-12-24 00:20:27,294 - ERROR - Authentication with username admin failed: invalid username or password

Comment 5 Brian Bouterse 2015-02-28 22:37:34 UTC

Moved to https://pulp.plan.io/issues/559