Merge pull request #137 from ajayyy/experimental

Add error message for 400 errors
This commit is contained in:
Ajay Ramachandran
2020-09-09 21:36:21 -04:00
committed by GitHub

View File

@@ -177,7 +177,7 @@ module.exports = async function postSkipSegments(req, res) {
//check if all correct inputs are here and the length is 1 second or more //check if all correct inputs are here and the length is 1 second or more
if (videoID == undefined || userID == undefined || segments == undefined || segments.length < 1) { if (videoID == undefined || userID == undefined || segments == undefined || segments.length < 1) {
//invalid request //invalid request
res.sendStatus(400); res.sendStatus(400).send("Parameters are not valid");
return; return;
} }
@@ -191,7 +191,7 @@ module.exports = async function postSkipSegments(req, res) {
for (let i = 0; i < segments.length; i++) { for (let i = 0; i < segments.length; i++) {
if (segments[i] === undefined || segments[i].segment === undefined || segments[i].category === undefined) { if (segments[i] === undefined || segments[i].segment === undefined || segments[i].category === undefined) {
//invalid request //invalid request
res.sendStatus(400); res.sendStatus(400).send("One of your segments are invalid");
return; return;
} }
@@ -206,7 +206,7 @@ module.exports = async function postSkipSegments(req, res) {
if (isNaN(startTime) || isNaN(endTime) if (isNaN(startTime) || isNaN(endTime)
|| startTime === Infinity || endTime === Infinity || startTime < 0 || startTime >= endTime) { || startTime === Infinity || endTime === Infinity || startTime < 0 || startTime >= endTime) {
//invalid request //invalid request
res.sendStatus(400); res.sendStatus(400).send("One of your segments times are invalid (too short, startTime before endTime, etc.)");
return; return;
} }