Skip to content

Commit

Permalink
use GCP options in integration if available
Browse files Browse the repository at this point in the history
  • Loading branch information
lobsterkatie committed Oct 19, 2022
1 parent 4f593a5 commit 43febe9
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion packages/node/src/integrations/requestdata.ts
Expand Up @@ -119,10 +119,11 @@ export class RequestData implements Integration {
}

// The Express request handler takes a similar `include` option to that which can be passed to this integration.
// If passed there, we store it in `sdkProcessingMetadata`. TODO(v8): Force express people to use this
// If passed there, we store it in `sdkProcessingMetadata`. TODO(v8): Force express and GCP people to use this
// integration, so that all of this passing and conversion isn't necessary
const addRequestDataOptions =
sdkProcessingMetadata.requestDataOptionsFromExpressHandler ||
sdkProcessingMetadata.requestDataOptionsFromGCPWrapper ||
convertReqDataIntegrationOptsToAddReqDataOpts(this._options);

const processedEvent = this._addRequestData(event, req, addRequestDataOptions);
Expand Down

0 comments on commit 43febe9

Please sign in to comment.