Skip to content

Commit 1881f51

Browse files
authored
Merge pull request #10 from wallymathieu/flohdot_master
Add escaping for bad platform string.
2 parents 5f703eb + f37116a commit 1881f51

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/nuts.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ Nuts.prototype.onDownload = function(req, res, next) {
195195
}
196196

197197
if (!asset) {
198-
res.status(400).send("No download available for platform "+platform+" for version "+version.tag+" ("+(channel || "beta")+")");
198+
res.status(400).send("No download available for platform "+_.escape(platform)+" for version "+version.tag+" ("+(channel || "beta")+")");
199199
return;
200200
}
201201

@@ -215,7 +215,7 @@ Nuts.prototype.onUpdateRedirect = function(req, res, next) {
215215
if (!req.query.version) throw new Error('Requires "version" parameter');
216216
if (!req.query.platform) throw new Error('Requires "platform" parameter');
217217

218-
return res.redirect('/update/'+req.query.platform+'/'+req.query.version);
218+
return res.redirect('/update/'+_.escape(req.query.platform)+'/'+_.escape(req.query.version));
219219
})
220220
.fail(next);
221221
};

0 commit comments

Comments
 (0)