Validate keypair create request body

Check if keypair create request body has manditory keys like keypair, name
and raise HTTPBadRequest if the required keys are missing.

Fixes bug 1049619

Change-Id: Ia6129e1b04e910fbea81b62b921a228aafa32f64

nova/api/openstack/compute/contrib/keypairs.py

@@ -70,8 +70,13 @@ def create(self, req, body):
 
         context = req.environ['nova.context']
         authorize(context)
-        params = body['keypair']
-        name = params['name']
+
+        try:
+            params = body['keypair']
+            name = params['name']
+        except KeyError:
+            msg = _("Invalid request body")
+            raise webob.exc.HTTPBadRequest(explanation=msg)
 
         try:
             if 'public_key' in params:

nova/tests/api/openstack/compute/contrib/test_keypairs.py

@@ -295,6 +295,18 @@ def test_detail_servers(self):
             self.assertTrue('key_name' in server_dict)
             self.assertEquals(server_dict['key_name'], '')
 
+    def test_keypair_create_with_invalid_keypairBody(self):
+        body = {'alpha': {'name': 'create_test'}}
+        req = webob.Request.blank('/v1.1/fake/os-keypairs')
+        req.method = 'POST'
+        req.body = jsonutils.dumps(body)
+        req.headers['Content-Type'] = 'application/json'
+        res = req.get_response(fakes.wsgi_app())
+        res_dict = jsonutils.loads(res.body)
+        self.assertEqual(res.status_int, 400)
+        self.assertEqual(res_dict['badRequest']['message'],
+                         "Invalid request body")
+
 
 class KeypairsXMLSerializerTest(test.TestCase):
     def setUp(self):