fix(pii): Allow scrubbing of frame.module and culprit

CHANGELOG.md

@@ -7,6 +7,7 @@
 - Add support for attaching Sentry event payloads in Unreal crash reports by adding `__sentry` game data entries. ([#715](https://github.com/getsentry/relay/pull/715))
 - Support chunked formdata keys for event payloads on the Minidump endpoint. Since crashpad has a limit for the length of custom attributes, the sentry event payload can be split up into `sentry__1`, `sentry__2`, etc. ([#721](https://github.com/getsentry/relay/pull/721))
 - Add periodic re-authentication with the upstream server, previously there was only one initial authentication. ([#731](https://github.com/getsentry/relay/pull/731))
+- The module attribute on stack frames (`$frame.module`) and the (usually serverside-generated) attribute `culprit` can now be scrubbed with advanced data scrubbing. ([#744](https://github.com/getsentry/relay/pull/744))
 
 **Internal**:
 

relay-general/src/protocol/event.rs

@@ -220,7 +220,7 @@ pub struct Event {
     pub fingerprint: Annotated<Fingerprint>,
 
     /// Custom culprit of the event.
-    #[metastructure(max_chars = "culprit")]
+    #[metastructure(max_chars = "culprit", pii = "maybe")]
     pub culprit: Annotated<String>,
 
     /// Transaction name of the event.

relay-general/src/protocol/stacktrace.rs

@@ -53,7 +53,7 @@ pub struct Frame {
     ///
     /// Note that this might also include a class name if that is something the
     /// language natively considers to be part of the stack (for instance in Java).
-    #[metastructure(skip_serialization = "empty")]
+    #[metastructure(skip_serialization = "empty", pii = "maybe")]
     // TODO: Cap? This can be a FS path or a dotted path
     pub module: Annotated<String>,