From 353642c870e569dfc251a5986a1586c8ce1b8450 Mon Sep 17 00:00:00 2001
From: Sheogorath
Date: Wed, 26 Sep 2018 16:00:01 +0200
Subject: 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>
---
 lib/response.js | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

(limited to 'lib')

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
-- 
cgit v1.2.3