fix(gatsby): Fix OOM from telemetry storing too much (#22752)

gatsbyjs · Apr 3, 2020 · a7281c2 · a7281c2
1 parent 5b35acc
commit a7281c2
Showing 1 changed file with 15 additions and 3 deletions.
diff --git a/packages/gatsby/src/query/graphql-runner.ts b/packages/gatsby/src/query/graphql-runner.ts
@@ -1,3 +1,4 @@
+import crypto from "crypto"
 import {
   parse,
   validate,
@@ -164,9 +165,20 @@ export default class GraphQLRunner {
       if (typeof statsQuery !== `string`) {
         statsQuery = statsQuery.body
       }
-      this.stats.uniqueOperations.add(`${statsQuery}${JSON.stringify(context)}`)
-
-      this.stats.uniqueQueries.add(statsQuery)
+      this.stats.uniqueOperations.add(
+        crypto
+          .createHash(`sha1`)
+          .update(statsQuery)
+          .update(JSON.stringify(context))
+          .digest(`hex`)
+      )
+
+      this.stats.uniqueQueries.add(
+        crypto
+          .createHash(`sha1`)
+          .update(statsQuery)
+          .digest(`hex`)
+      )
     }
 
     const document = this.parse(query)