)]}'
{"nova/api/openstack/compute/contrib/keypairs.py":[{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"c737a9f920bd951ed83de9333f8f74cee9dbef23","unresolved":false,"context_lines":[{"line_number":75,"context_line":"            params \u003d body[\u0027keypair\u0027]"},{"line_number":76,"context_line":"            name \u003d params[\u0027name\u0027]"},{"line_number":77,"context_line":"        except KeyError:"},{"line_number":78,"context_line":"            msg \u003d _(\"Invalid request body\")"},{"line_number":79,"context_line":"            raise webob.exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAALH%2F%2FUfc%3D","line":78,"updated":"2012-09-21 15:36:22.000000000","message":"Can we make this message even more explicit? Such as, \"keypair\" or \"name\" not in request body. Or possibly have two separate tests, First test if \"keypair is in body and respond with message. Invalid request body. \"Keypair\" not found.\n\nThen the second test for \"name\" and Respond with \"Invalid request body. Name not found. I know in some places in the API the error responses are more explicit when possible. I had checked in some code recently in which I was asked to be more explicit.","commit_id":"760e59b8632edd8cbf395289e24f380ffd06f047"}]}