Add a better maxfiles message, a little tweak for TOR, and add it to banner uploads closes #225

merge-requests/208/head
Thomas Lynch 4 years ago
parent 24574862a2
commit 73fa1b313e
  1. 11
      helpers/filemiddlewares.js

@ -12,7 +12,15 @@ const { globalLimits, debugLogs, filterFileNames, spaceFileNameReplacement } = r
, numFilesUploadLimitFunction = (req, res, next) => {
return dynamicResponse(req, res, 400, 'message', {
'title': 'Too many files',
'message': 'You sent too many files in one request',
'message': res.locals.board ? `Max files per post ${res.locals.board.settings.maxFiles < globalLimits.postFiles.max ? 'on this board ' : ''}is ${res.locals.board.settings.maxFiles}`
: `Max files per request is ${globalLimits.postFiles.max}`, //because of difference in TOR body parsing, we dont populate res.locals.board at this point. something to address later.
'redirect': req.headers.referer
});
}
, numBannersUploadLimitFunction = (req, res, next) => {
return dynamicResponse(req, res, 400, 'message', {
'title': 'Too many files',
'message': `Max banners per request is ${globalLimits.bannerFiles.max}`,
'redirect': req.headers.referer
});
}
@ -48,6 +56,7 @@ module.exports = {
fileSize: globalLimits.bannerFilesSize.max,
files: globalLimits.bannerFiles.max
},
numFilesLimitHandler: numBannersUploadLimitFunction,
limitHandler: uploadLimitFunction,
useTempFiles: true,
tempFileDir: __dirname+'/../tmp/'

Loading…
Cancel
Save