Merge pull request #5189 from darrellwarde/fix/remove-log-params

Remove params from transaction metadata
neo4j · May 31, 2024 · c8c65c1 · c8c65c1
2 parents c637683 + 7046091
commit c8c65c1
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 16 deletions.
diff --git a/.changeset/cold-tables-trade.md b/.changeset/cold-tables-trade.md
@@ -0,0 +1,5 @@
+---
+"@neo4j/graphql": patch
+---
+
+Remove parameters from transaction metadata
diff --git a/packages/graphql/src/classes/Executor.ts b/packages/graphql/src/classes/Executor.ts
@@ -17,35 +17,35 @@
  * limitations under the License.
  */
 
+import Debug from "debug";
+import type { GraphQLResolveInfo } from "graphql";
+import { print } from "graphql";
 import type {
     Driver,
+    ManagedTransaction,
     QueryResult,
+    Result,
     Session,
+    SessionConfig,
     SessionMode,
     Transaction,
-    SessionConfig,
-    ManagedTransaction,
-    Result,
 } from "neo4j-driver";
 import { Neo4jError } from "neo4j-driver";
-import Debug from "debug";
-import environment from "../environment";
-import {
-    Neo4jGraphQLAuthenticationError,
-    Neo4jGraphQLConstraintValidationError,
-    Neo4jGraphQLForbiddenError,
-    Neo4jGraphQLRelationshipValidationError,
-} from "./Error";
 import {
     AUTH_FORBIDDEN_ERROR,
     AUTH_UNAUTHENTICATED_ERROR,
     DEBUG_EXECUTE,
     RELATIONSHIP_REQUIREMENT_PREFIX,
 } from "../constants";
-import type { CypherQueryOptions } from "../types";
-import type { GraphQLResolveInfo } from "graphql";
-import { print } from "graphql";
 import { debugCypherAndParams } from "../debug/debug-cypher-and-params";
+import environment from "../environment";
+import type { CypherQueryOptions } from "../types";
+import {
+    Neo4jGraphQLAuthenticationError,
+    Neo4jGraphQLConstraintValidationError,
+    Neo4jGraphQLForbiddenError,
+    Neo4jGraphQLRelationshipValidationError,
+} from "./Error";
 
 const debug = Debug(DEBUG_EXECUTE);
 
@@ -72,7 +72,6 @@ type TransactionConfig = {
         type: "system" | "user-direct" | "user-action" | "user-transpiled";
         source?: {
             query: string;
-            params: Record<string, unknown>;
         };
     };
 };
@@ -212,7 +211,6 @@ export class Executor {
                     [Object.values(info.fragments).map((fragment) => print(fragment)), print(info.operation)].join(
                         "\n\n"
                     ),
-                params: info.variableValues,
             };
 
             transactionConfig.metadata.source = source;