ben.carrasco/jambonz-feature-server
1
0
Fork 0
mirror of https://github.com/jambonz/jambonz-feature-server.git synced 2025-12-20 08:40:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix #493: remove nulls from cerateCall payload in order not to trip up validation (#494)

Browse Source
This commit is contained in:
Dave Horton
2023-10-23 10:58:28 -04:00
committed by GitHub
parent cf0f4d405f
commit 168b4dc051
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lib/http-routes/api/create-call.js
View File

@@ -15,14 +15,22 @@ const dbUtils = require('../../utils/db-utils');
const { mergeSdpMedia, extractSdpMedia } = require('../../utils/sdp-utils'); const { mergeSdpMedia, extractSdpMedia } = require('../../utils/sdp-utils');
const { createCallSchema } = require('../schemas/create-call'); 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('/', router.post('/',
removeNulls,
createCallSchema, createCallSchema,
async(req, res) => { async(req, res) => {
const {logger} = req.app.locals;
const errors = validationResult(req); const errors = validationResult(req);
if (!errors.isEmpty()) { if (!errors.isEmpty()) {
logger.info({errors: errors.array()}, 'POST /Calls: validation errors');
return res.status(400).json({ errors: errors.array() }); return res.status(400).json({ errors: errors.array() });
} }
const {logger} = req.app.locals;
const accountSid = req.body.account_sid; const accountSid = req.body.account_sid;
const {srf} = require('../../..'); const {srf} = require('../../..');