Issue #316
closedTraceback is raised if required key is not provided during consumer registration
Description
Traceback is raised if required key( consumer id) is not provided during consumer registration.
Instead 400 should be raised.
$ curl -i -H "Accept: application/json" -H "WebFrameworkSwitch: webpy" -X POST -k -u admin:admin -d '{}' "https://localhost/pulp/api/v2/consumers/"
HTTP/1.1 500 Internal Server Error
Date: Thu, 26 Feb 2015 09:18:05 GMT
Server: Apache/2.4.10 (Fedora) OpenSSL/1.0.1e-fips mod_wsgi/3.5 Python/2.7.5
Content-Encoding: utf-8
Content-Length: 4905
Connection: close
Content-Type: application/json
{"http_request_method": "POST", "exception": ["TypeError: expected string or buffer\n"], "error_message": "expected string or buffer", "_href": "/pulp/api/v2/consumers/", "http_status": 500, "traceback": [" File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/middleware/exception.py\", line 46, in call\n return self.app(environ, start_response)\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/middleware/postponed.py\", line 55, in call\n return self.app(environ, start_response)\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 279, in wsgi\n result = self.handle_with_processors()\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 18, in _handle_with_processors\n return process(self.processors)\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 15, in process\n return p(lambda: process(processors))\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 566, in processor\n return handler()\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 15, in <lambda>\n return p(lambda: process(processors))\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 15, in process\n return p(lambda: process(processors))\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 581, in processor\n result = handler()\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 15, in <lambda>\n return p(lambda: process(processors))\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 17, in process\n return self.handle()\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 230, in handle\n return self._delegate(fn, self.fvars, args)\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 422, in _delegate\n return f()\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 430, in <lambda>\n f = lambda: self._delegate_sub_application(pat, what)\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 455, in _delegate_sub_application\n return app.handle_with_processors()\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 18, in _handle_with_processors\n return process(self.processors)\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 15, in process\n return p(lambda: process(processors))\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 566, in processor\n return handler()\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 15, in <lambda>\n return p(lambda: process(processors))\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 15, in process\n return p(lambda: process(processors))\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 581, in processor\n result = handler()\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 15, in <lambda>\n return p(lambda: process(processors))\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/application.py\", line 17, in process\n return self.handle()\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 230, in handle\n return self._delegate(fn, self.fvars, args)\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 405, in _delegate\n return handle_class(f)\n", " File \"/usr/lib/python2.7/site-packages/web/application.py\", line 396, in handle_class\n return tocall(*args)\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/controllers/decorators.py\", line 241, in _auth_decorator\n return _verify_auth(self, operation, super_user_only, method, *args, **kwargs)\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/controllers/decorators.py\", line 195, in _verify_auth\n value = method(self, *args, **kwargs)\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/webservices/controllers/consumers.py\", line 87, in POST\n rsa_pub=rsa_pub)\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/managers/consumer/cud.py\", line 59, in register\n if not is_consumer_id_valid(consumer_id):\n", " File \"/home/ipanova/pulp_development/pulp/server/pulp/server/managers/consumer/cud.py\", line 279, in is_consumer_id_valid\n result = _CONSUMER_ID_REGEX.match(id) is not None\n"]}
Updated by ipanova@redhat.com about 9 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to ipanova@redhat.com
Updated by bmbouter about 9 years ago
- Triaged changed from No to Yes
- Severity set to Medium
Updated by ipanova@redhat.com about 9 years ago
- Status changed from ASSIGNED to POST
Updated by ipanova@redhat.com about 9 years ago
- Status changed from POST to MODIFIED
Updated by dkliban@redhat.com almost 9 years ago
- Status changed from MODIFIED to 5
Updated by igulina@redhat.com almost 9 years ago
- Status changed from 5 to 6
on f20
>> rpm -qa pulp-server
pulp-server-2.7.0-0.3.beta.fc20.noarch
>> pulp-admin login -u admin -p admin
Successfully logged in. Session certificate will expire at Jun 25 08:28:30 2015
GMT.
>> curl -i -H "Accept: application/json" -H "WebFrameworkSwitch: webpy" -X POST -k -u admin:admin -d '{}' "https://localhost/pulp/api/v2/consumers/"
HTTP/1.1 400 BAD REQUEST
Date: Thu, 18 Jun 2015 08:28:42 GMT
Server: Apache/2.4.10 (Fedora) OpenSSL/1.0.1e-fips mod_wsgi/3.5 Python/2.7.5
Content-Encoding: utf-8
Content-Length: 359
Connection: close
Content-Type: application/json
{"http_request_method": "POST", "exception": null, "error_message": "Missing values for: ['id']", "_href": "/pulp/api/v2/consumers/", "http_status": 400, "error": {"code": "PLP0016", "data": {"property_names": ["id"], "properties": "['id']"}, "description": "Missing values for: ['id']", "sub_errors": []}, "missing_property_names": ["id"], "traceback": null}
on rhel6
>> rpm -qa pulp-server
pulp-server-2.7.0-0.3.beta.el6.noarch
>> curl -i -H "Accept: application/json" -H "WebFrameworkSwitch: webpy" -X POST -k -u admin:admin -d '{}' "https://localhost/pulp/api/v2/consumers/"
HTTP/1.1 400 BAD REQUEST
Date: Thu, 18 Jun 2015 08:35:52 GMT
Server: Apache/2.2.15 (Red Hat)
Content-Encoding: utf-8
Content-Length: 359
Connection: close
Content-Type: application/json
{"http_request_method": "POST", "exception": null, "error_message": "Missing values for: ['id']", "_href": "/pulp/api/v2/consumers/", "http_status": 400, "error": {"code": "PLP0016", "data": {"property_names": ["id"], "properties": "['id']"}, "description": "Missing values for: ['id']", "sub_errors": []}, "missing_property_names": ["id"], "traceback": null}
Updated by amacdona@redhat.com over 8 years ago
- Status changed from 6 to CLOSED - CURRENTRELEASE
Updated by bmbouter almost 4 years ago
- Category deleted (
14)
We are removing the 'API' category per open floor discussion June 16, 2020.