Fixed #31: moved bodyparsing to allow upload of non-json

app/templates/node-server/node-app.js

@@ -4,7 +4,6 @@
 var express = require('express');
 var expressSession = require('express-session');
 var app = express();
-var bodyParser = require('body-parser');
 var logger = require('morgan');
 var port = process.env.PORT || 8001;
 var four0four = require('./utils/404')();
@@ -18,13 +17,10 @@ app.use(expressSession({
   resave: true
 }));
 
-app.use(bodyParser.urlencoded({extended: true}));
-app.use(bodyParser.json());
 app.use(logger('dev'));
 
-app.use('/api', require('./routes'));
-
 app.use('/v1', require('./proxy'));
+app.use('/api', require('./routes'));
 
 app.use('/create', express.static('./build/index.html'));
 app.use('/profile', express.static('./build/index.html'));

app/templates/node-server/routes.js

@@ -3,6 +3,7 @@
 'use strict';
 
 var router = require('express').Router();
+var bodyParser = require('body-parser');
 var four0four = require('./utils/404')();
 var http = require('http');
 var config = require('../gulp.config')();
@@ -15,6 +16,10 @@ var options = {
   defaultPass: process.env.ML_APP_PASS || config.marklogic.password
 };
 
+// [GJo] (#31) Moved bodyParsing inside routing, otherwise it might try to parse uploaded binaries as json..
+router.use(bodyParser.urlencoded({extended: true}));
+router.use(bodyParser.json());
+
 router.get('/user/status', function(req, res) {
   var headers = req.headers;
   noCache(res);