Allow Session Replay "replaysOnErrorSampleRate" variable to accept an array of possible logging levels

[michaelcreatesstuff]

Problem Statement

I want to not capture replays for all errors, only errors with a logging level of "error" or "fatal"

I am using Sentry to log events with level "debug". I don't want these included in Session replay

Solution Brainstorm

Allow the user to provide an array of logging levels to include in Session Replay's "replaysOnErrorSampleRate" variable config.

Default this to all logging levels if no value is provided

Thank you!

[mydea]

Hey,

thanks for writing in!

I think a better way to realise this is to allow to put a property on the event telling replay to ignore this.

E.g. something like this:

Sentry.init({
  beforeSend(event) {
    // if this should not go to replay:
    event.IGNORE_REPLAY = true;
  }
});

cc @billyvg WDYT?

[michaelcreatesstuff]

@mydea sure, so if we were capturing an exception, it would look like this?

   SentryHelper_captureException(error, {
        level: "debug",
        ignore_replay: true
      });

[mydea]

@mydea sure, so if we were capturing an exception, it would look like this?

   SentryHelper_captureException(error, {
        level: "debug",
        ignore_replay: true
      });

If we go with my idea above, you'd have to add an event processor or a beforeSend callback and add a specified property to each event you want to ignore for replay.

[michaelcreatesstuff]

If we go with my idea above, you'd have to add an event processor or a beforeSend callback and add a specified property to each event you want to ignore for replay.

Ok, let me make sure I understand correctly.

First I would capture an exception like this:

Sentry.captureException(error, {
  level: "debug",
  extra: {
    ignore_replay: true
  }
}

then in the beforeSend callback I would add this

Sentry.init({
  dsn: "https://**.ingest.sentry.io/**",
  beforeSend(event) {
    if (event.extra.ignore_replay) {
      event.IGNORE_REPLAY = true;
    }
    return event;
  },
});

Is that correct?

[mydea]

Yeah, something like this (although the exact API & naming is still open) - but API-wise, that could work I believe!

[michaelcreatesstuff]

@mydea thank you, that's great to hear. This would be really helpful for us, is this something the Sentry team could implement in the near future?

[bruno-garcia]

Do we take hint as a second argument on beforeSend? If so that'd be a better place to set a flag to avoid kicking off the reply.

[mydea]

Hey!

In 7.78.0, you can now configure a callback to replay to solve this:

new Replay({
  beforeErrorSampling(event) {
    // return true to sample for this error, or false to not sample
  }
})

[michaelcreatesstuff]

Hey!

In 7.78.0, you can now configure a callback to replay to solve this:

new Replay({
  beforeErrorSampling(event) {
    // return true to sample for this error, or false to not sample
  }
})

Thank you so much for the quick turnaround! Much appreciated