Issue #559
closedTraceback is shown in consumer.log if pulp-consumer experiences an authentication failure
Description
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
+ This bug was cloned from Bugzilla Bug #1148919 +
Updated by rbarlow about 9 years ago
Austin, this is low hanging fruit if you want an easy BZ.
+ This comment was cloned from Bugzilla #1148919 comment 1 +
Updated by amacdona@redhat.com about 9 years ago
Fixed by https://github.com/pulp/pulp/pull/1190
+ This comment was cloned from Bugzilla #1148919 comment 2 +
Updated by cduryee about 9 years ago
fixed in pulp 2.6.0-0.2.beta
+ This comment was cloned from Bugzilla #1148919 comment 3 +
Updated by igulina@redhat.com about 9 years ago
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
+ This comment was cloned from Bugzilla #1148919 comment 4 +
Updated by rbarlow about 9 years ago
- Status changed from 6 to CLOSED - CURRENTRELEASE