Skip to content

Commit

Permalink
fix(mongo): Remove aggregate from operations whose arguments get se…
Browse files Browse the repository at this point in the history
…rialized (#3102)

The arguments don't serialize well, because they're too nested/complex, and also potentially quite large. See PR for examples.
  • Loading branch information
lobsterkatie authored Dec 4, 2020
1 parent 1ab3b04 commit b90dfd6
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion packages/tracing/src/integrations/mongo.ts
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,8 @@ const OPERATIONS = [
const OPERATION_SIGNATURES: {
[op in Operation]?: string[];
} = {
aggregate: ['pipeline'],
// aggregate intentionally not included because `pipeline` arguments are too complex to serialize well
// see https://github.com/getsentry/sentry-javascript/pull/3102
bulkWrite: ['operations'],
countDocuments: ['query'],
createIndex: ['fieldOrSpec'],
Expand Down

0 comments on commit b90dfd6

Please sign in to comment.