fix #493: remove nulls from cerateCall payload in order not to trip u…

…p validation

lib/http-routes/api/create-call.js

@@ -15,14 +15,22 @@ const dbUtils = require('../../utils/db-utils');
 const { mergeSdpMedia, extractSdpMedia } = require('../../utils/sdp-utils');
 const { createCallSchema } = require('../schemas/create-call');
 
+const removeNullProperties = (obj) => (Object.keys(obj).forEach((key) => obj[key] === null && delete obj[key]), obj);
+const removeNulls = (req, res, next) => {
+  req.body = removeNullProperties(req.body);
+  next();
+};
+
 router.post('/',
+  removeNulls,
   createCallSchema,
   async(req, res) => {
+    const {logger} = req.app.locals;
     const errors = validationResult(req);
     if (!errors.isEmpty()) {
+      logger.info({errors: errors.array()}, 'POST /Calls: validation errors');
       return res.status(400).json({ errors: errors.array() });
     }
-    const {logger} = req.app.locals;
     const accountSid = req.body.account_sid;
     const {srf} = require('../../..');