Merge pull request #8031 from slagiewka/migration_middleware_return

docs(migration): return after sending 400

Author: Meschreiber

.changeset/thick-seals-tan.md

@@ -0,0 +1,5 @@
+---
+'@apollo/server': patch
+---
+
+Add return after sending 400 response in doubly escaped JSON parser middleware

docs/source/migration.mdx

@@ -1121,6 +1121,7 @@ app.use((req, res, next) => {
     } catch (e) {
       // https://github.com/graphql/graphql-over-http/blob/main/spec/GraphQLOverHTTP.md#json-parsing-failure
       res.status(400).send(e instanceof Error ? e.message : e);
+      return;
     }
   }
   next();

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

@@ -163,6 +163,7 @@ it('supporting doubly-encoded variables example from migration guide', async ()
       } catch (e) {
         // https://github.com/graphql/graphql-over-http/blob/main/spec/GraphQLOverHTTP.md#json-parsing-failure
         res.status(400).send(e instanceof Error ? e.message : e);
+        return;
       }
     }
     next();