Merge pull request #114 from AlexandreBeaurain/master

fix: bad cookies can crash process
AdFabConnect · Jan 9, 2017 · 68085c8 · 68085c8
2 parents 8b7e75a + 13f3fa1
commit 68085c8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/server/routes/get-main.js b/src/server/routes/get-main.js
@@ -48,7 +48,7 @@ function renderAbeAdmin(EditorVariables, obj, filePath) {
   var editorWidth = '33%'
   EditorVariables.express.req.headers && EditorVariables.express.req.headers.cookie && EditorVariables.express.req.headers.cookie.split(';').forEach(function(cookie) {
     var parts = cookie.match(/(.*?)=(.*)$/)
-    if(parts[1] === 'editorWidth') editorWidth = parts[2]
+    if ( parts && ( parts.length > 2 ) && ( parts[1] === 'editorWidth' ) ) editorWidth = parts[2]
   })
 
   EditorVariables.pageHtml = pageHtml
@@ -173,4 +173,4 @@ var route = function(req, res, next) {
   })
 }
 
-export default route
+export default route