Null value in extra map causes event to get silently dropped

[harlandduman]

Integration

sentry-android

Build System

Gradle

AGP Version

7.4.2

Proguard

Enabled

Version

6.9.2

Steps to Reproduce

I'm using React Native, but manually initializing Sentry on Android (not sure if the manual initialization is relevant or not)

Capture exception in Javascript, passing in an extra map with a null value in it, e.g.

Sentry.withScope((scope) => {
  scope.setExtras({someKey: null});
  Sentry.captureException(error)
}

Expected Result

The event should be processed successfully and sent to Sentry.

Actual Result

There's an error on the Android side when deserializing the event, and it gets silently dropped. I also would expect that an error during processing would get sent to Sentry since it seems like that's what happens in the Javascript library, but maybe I'm wrong about that.

It looks like this line

sentry-java/sentry/src/main/java/io/sentry/SentryBaseEvent.java

Line 447 in 308cdb2

baseEvent.extra = CollectionUtils.newConcurrentHashMap(deserializedExtra);

is the issue, since ConcurrentHashMaps don't support null values

[romtsn]

@harlandduman thanks for the report, we'll look into this soon!