restore: only take non-empty backupId

This commit is contained in:
Girish Ramakrishnan
2019-12-05 21:15:09 -08:00
parent c5e5bb90e3
commit 3926efd153
2 changed files with 6 additions and 7 deletions

View File

@@ -377,10 +377,9 @@ function restoreApp(req, res, next) {
debug('Restore app id:%s', req.params.id);
if (!('backupId' in req.body)) return next(new HttpError(400, 'backupId is required'));
if (data.backupId !== null && typeof data.backupId !== 'string') return next(new HttpError(400, 'backupId must be string or null'));
if (!data.backupId || typeof data.backupId !== 'string') return next(new HttpError(400, 'backupId must be non-empty string'));
apps.restore(req.params.id, data, auditSource.fromRequest(req), function (error, result) {
apps.restore(req.params.id, data.backupId, auditSource.fromRequest(req), function (error, result) {
if (error) return next(BoxError.toHttpError(error));
next(new HttpSuccess(202, { taskId: result.taskId }));