summaryrefslogtreecommitdiff
path: root/lib/response.js
diff options
context:
space:
mode:
authorSheogorath2018-09-26 16:00:01 +0200
committerSheogorath2018-09-26 16:08:24 +0200
commit353642c870e569dfc251a5986a1586c8ce1b8450 (patch)
tree85d063e50a7fbd7209b0bf8930d7e61ee33cabba /lib/response.js
parentbdf897d31c7eb5f819807c6910ce1aea84cbf7ee (diff)
Fix document length limit on post
We recently introduced a new way to create notes using a post requeest to the `/new` endpoint. This is not limited in size, other than pasting a note in the editor. This patch should enforce this limit also on this way. Signed-off-by: Sheogorath <sheogorath@shivering-isles.com>
Diffstat (limited to '')
-rw-r--r--lib/response.js10
1 files changed, 9 insertions, 1 deletions
diff --git a/lib/response.js b/lib/response.js
index 295f91d6..8133b1a0 100644
--- a/lib/response.js
+++ b/lib/response.js
@@ -32,6 +32,9 @@ var response = {
errorBadRequest: function (res) {
responseError(res, '400', 'Bad Request', 'something not right.')
},
+ errorTooLong: function (res) {
+ responseError(res, '413', 'Payload Too Large', 'Shorten your note!')
+ },
errorInternalError: function (res) {
responseError(res, '500', 'Internal Error', 'wtf.')
},
@@ -145,7 +148,12 @@ function responseCodiMD (res, note) {
function newNote (req, res, next) {
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, '')
if (req.isAuthenticated()) {
owner = req.user.id