depau/HackMD
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #966 from SISheogorath/fix/documentLength

Browse source 
Fix document length limit on post
This commit is contained in:
Christoph (Sheogorath) Kern 2018-09-27 20:10:50 +02:00 committed by GitHub
commit 55f7568985
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
lib/response.js
View file

@ -32,6 +32,9 @@ var response = {
errorBadRequest: function (res) { errorBadRequest: function (res) {
responseError(res, '400', 'Bad Request', 'something not right.') responseError(res, '400', 'Bad Request', 'something not right.')
}, },
errorTooLong: function (res) {
responseError(res, '413', 'Payload Too Large', 'Shorten your note!')
},
errorInternalError: function (res) { errorInternalError: function (res) {
responseError(res, '500', 'Internal Error', 'wtf.') responseError(res, '500', 'Internal Error', 'wtf.')
}, },
@ -145,7 +148,12 @@ function responseCodiMD (res, note) {
function newNote (req, res, next) { function newNote (req, res, next) {
var owner = null var owner = null
var body = req.body ? req.body : '' var body = ''
if (req.body && req.body.length > config.documentMaxLength) {
return response.errorTooLong(res)
} else {
body = req.body
}
body = body.replace(/[\r]/g, '') body = body.replace(/[\r]/g, '')
if (req.isAuthenticated()) { if (req.isAuthenticated()) {
owner = req.user.id owner = req.user.id