make options optional

Author: lobsterkatie

packages/node/src/integrations/requestdata.ts

@@ -10,7 +10,7 @@ type RequestDataIntegrationOptions = {
   /**
    * Controls what data is pulled from the request and added to the event
    */
-  include: {
+  include?: {
     cookies?: boolean;
     data?: boolean;
     headers?: boolean;
@@ -27,15 +27,15 @@ type RequestDataIntegrationOptions = {
   };
 
   /** Whether to identify transactions by parameterized path, parameterized path with method, or handler name */
-  transactionNamingScheme: TransactionNamingScheme;
+  transactionNamingScheme?: TransactionNamingScheme;
 
   /**
    * Function for adding request data to event. Defaults to `addRequestDataToEvent` from `@sentry/node` for now, but
    * left injectable so this integration can be moved to `@sentry/core` and used in browser-based SDKs in the future.
    *
    * @hidden
    */
-  addRequestData: typeof addRequestDataToEvent;
+  addRequestData?: typeof addRequestDataToEvent;
 };
 
 const DEFAULT_OPTIONS = {
@@ -65,12 +65,12 @@ export class RequestData implements Integration {
    */
   public name: string = RequestData.id;
 
-  private _options: RequestDataIntegrationOptions;
+  private _options: Required<RequestDataIntegrationOptions>;
 
   /**
    * @inheritDoc
    */
-  public constructor(options: Partial<RequestDataIntegrationOptions> = {}) {
+  public constructor(options: RequestDataIntegrationOptions = {}) {
     this._options = {
       ...DEFAULT_OPTIONS,
       ...options,
@@ -142,7 +142,7 @@ export class RequestData implements Integration {
 /** Convert this integration's options to match what `addRequestDataToEvent` expects */
 /** TODO: Can possibly be deleted once https://github.com/getsentry/sentry-javascript/issues/5718 is fixed */
 function convertReqDataIntegrationOptsToAddReqDataOpts(
-  integrationOptions: RequestDataIntegrationOptions,
+  integrationOptions: Required<RequestDataIntegrationOptions>,
 ): AddRequestDataToEventOptions {
   const { transactionNamingScheme } = integrationOptions;
   const { ip, user, ...requestOptions } = integrationOptions.include;