Merge pull request #7 from TwilioDevEd/DEVED-4699-missing-body-parser

[DEVED-4699] Add a comment on missing body-parser

app.js

@@ -10,6 +10,13 @@ if (app.get('env') === 'development') {
 }
 
 app.use(logger('dev'));
+
+// Since Express v4.16.0 it already includes a parser middleware (based on body-parser),
+// so no need to add the dependency: `const bodyParser = require('body-parser');`.
+
+// `require('body-parser')` may be needed in cases where additional functionality is required,
+// like `bodyParser.raw([options])` or `bodyParser.text([options])`.
+
 app.use(express.json());
 app.use(express.urlencoded({ extended: false }));
 

package.json

@@ -9,18 +9,18 @@
     "test_watch": "./node_modules/.bin/mocha -w test"
   },
   "dependencies": {
-    "eslint": "^6.8.0",
-    "eslint-config-airbnb-base": "^14.0.0",
-    "eslint-plugin-import": "^2.20.1",
-    "eslint-plugin-mocha": "^6.2.2",
+    "eslint": "^7.5.0",
+    "eslint-config-airbnb-base": "^14.2.0",
+    "eslint-plugin-import": "^2.22.0",
+    "eslint-plugin-mocha": "^7.0.1",
     "express": "~4.17.1",
-    "http-errors": "~1.7.3",
-    "mocha": "^7.0.1",
-    "morgan": "~1.9.1",
+    "http-errors": "~1.8.0",
+    "mocha": "^8.1.0",
+    "morgan": "~1.10.0",
     "should": "^13.2.3",
     "supertest": "^4.0.2",
-    "tracer": "^1.0.2",
-    "twilio": "^3.39.4"
+    "tracer": "^1.1.3",
+    "twilio": "^3.48.1"
   },
   "devDependencies": {
     "chai": "^4.2.0"

test/index.spec.js

@@ -6,7 +6,6 @@ require('chai');
 
 const supertest = require('supertest');
 
-
 function Session() {
   return supertest(require('../app')); // eslint-disable-line global-require
 }