Fix/use express built in json middleware (#7740)

Use built-in `express.json()` functionality instead of separate `body-parser` package. --------- Co-authored-by: Trevor Scheer <trevor.scheer@gmail.com>

Author: barnisanov

.changeset/honest-kangaroos-appear.md

@@ -0,0 +1,6 @@
+---
+'@apollo/server-integration-testsuite': patch
+'@apollo/server': patch
+---
+
+Uninstalled `body-parser` and used `express` built-in `body-parser` functionality instead(mainly the json middleware)

package-lock.json

@@ -51,7 +51,6 @@
  "@josephg/resolvable": "1.0.1",
  "@rollup/plugin-commonjs": "25.0.7",
  "@types/async-retry": "1.4.7",
- "@types/body-parser": "1.19.4",
  "@types/compression": "1.7.4",
  "@types/cors": "2.8.15",
  "@types/express": "4.17.20",
@@ -74,7 +73,6 @@
  "@typescript-eslint/parser": "5.62.0",
  "@vendia/serverless-express": "4.10.4",
  "awaiting": "3.0.0",
- "body-parser": "1.20.2",
  "codecov": "3.8.3",
  "compression": "1.7.4",
  "cors": "2.8.5",

package.json

@@ -29,11 +29,10 @@
  "@apollo/cache-control-types": "^1.0.3",
  "@apollo/client": "^3.6.9",
  "@apollo/server": "4.9.5",
- "@apollo/utils.keyvaluecache": "^2.1.0",
- "@apollo/utils.createhash": "^2.0.0",
  "@apollo/usage-reporting-protobuf": "^4.1.1",
+ "@apollo/utils.createhash": "^2.0.0",
+ "@apollo/utils.keyvaluecache": "^2.1.0",
  "@josephg/resolvable": "^1.0.1",
- "body-parser": "^1.20.0",
  "express": "^4.18.1",
  "graphql-http": "1.22.0",
  "graphql-tag": "^2.12.6",

packages/integration-testsuite/package.json

@@ -1,7 +1,6 @@
 import http from 'http';
 import { createHash } from '@apollo/utils.createhash';
 import express from 'express';
-import bodyParser from 'body-parser';
 import loglevel from 'loglevel';
 
 import { Report, Trace } from '@apollo/usage-reporting-protobuf';
@@ -874,7 +873,7 @@ export function defineIntegrationTestSuiteApolloServerTests(
  next();
  });
  this.app.use(
- bodyParser.raw({
+ express.raw({
  inflate: true,
  type: '*/*',
  }),

packages/integration-testsuite/src/apolloServerTests.ts

@@ -137,11 +137,10 @@
  "@apollo/utils.withrequired": "^2.0.0",
  "@graphql-tools/schema": "^9.0.0",
  "@josephg/resolvable": "^1.0.0",
- "@types/express-serve-static-core": "^4.17.30",
  "@types/express": "^4.17.13",
+ "@types/express-serve-static-core": "^4.17.30",
  "@types/node-fetch": "^2.6.1",
  "async-retry": "^1.2.1",
- "body-parser": "^1.20.0",
  "cors": "^2.8.5",
  "express": "^4.17.1",
  "loglevel": "^1.6.8",

packages/server/package.json

@@ -1,14 +1,13 @@
-import express from 'express';
+import express, { json } from 'express';
 import request from 'supertest';
 import compression, { filter as defaultFilter } from 'compression';
 import { ApolloServer, BaseContext } from '../../index.js';
 import { expressMiddleware } from '../../express4/index.js';
 import { it, expect } from '@jest/globals';
 import resolvable from '@josephg/resolvable';
 import cors from 'cors';
-import { json } from 'body-parser';
 
-it('gives helpful error if body-parser middleware is not installed', async () => {
+it('gives helpful error if json middleware is not installed', async () => {
  const server = new ApolloServer({ typeDefs: 'type Query {f: ID}' });
  await server.start();
  const app = express();
@@ -18,7 +17,7 @@ it('gives helpful error if body-parser middleware is not installed', async () =>
  await request(app)
  .post('/')
  .send({ query: '{hello}' })
- .expect(500, /forgot to set up the `body-parser`/);
+ .expect(500, /forgot to set up the `json` middleware/);
  await server.stop();
 });
 

packages/server/src/__tests__/express4/expressSpecific.test.ts

@@ -1,6 +1,5 @@
-import { json } from 'body-parser';
 import cors from 'cors';
-import express from 'express';
+import express, { json } from 'express';
 import http from 'http';
 import { ApolloServer, ApolloServerOptions, BaseContext } from '@apollo/server';
 import { expressMiddleware } from '@apollo/server/express4';

packages/server/src/__tests__/express4/integration.test.ts

@@ -1,6 +1,5 @@
-import { json } from 'body-parser';
 import cors from 'cors';
-import express from 'express';
+import express, { json } from 'express';
 import http from 'http';
 import { ApolloServer, ApolloServerOptions, BaseContext } from '@apollo/server';
 import { expressMiddleware } from '@apollo/server/express4';

packages/server/src/__tests__/express4/integrationServerless.test.ts

@@ -52,7 +52,7 @@ export function expressMiddleware<TContext extends BaseContext>(
  res.status(500);
  res.send(
  '`req.body` is not set; this probably means you forgot to set up the ' +
- '`body-parser` middleware before the Apollo Server middleware.',
+ '`json` middleware before the Apollo Server middleware.',
  );
  return;
  }