Codecov Report

Base: 80.10% // Head: 80.11% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (fff5493) compared to base (4796772).
Patch coverage: 100.00% of modified lines in pull request are covered.

@@             Coverage Diff              @@
##               main    #2246      +/-   ##
============================================
+ Coverage     80.10%   80.11%   +0.01%     
- Complexity     3430     3433       +3     
============================================
  Files           242      242              
  Lines         12758    12765       +7     
  Branches       1703     1703              
============================================
+ Hits          10220    10227       +7     
  Misses         1889     1889              
  Partials        649      649              

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Is there still a means to add global MDC tags?

@jeacott1 you can set a list of MDC tags that Sentry will pick up and convert to tags as shown in this sample:

I looks like we never documented this feature, created #2351 to do so.

Is the feature not working for you?

@adinauer the option used to exist due to my lost commit https://github.com/getsentry/sentry-java/pull/794/files
I didn't know there was now an alternative.
thanks!

@norwood how can this encoder be configured? If I add an encoder to my logback.xml, in Sentry I see an <unlabeled event>.

i have only configured manually, not with logback.xml, and i'm not familiar with the xml configuration path.

@norwood thanks for reply. It looks like this:

<appender name="SENTRY" class="io.sentry.logback.SentryAppender">
  <encoder class="ch.qos.logback.core.encoder.LayoutWrappingEncoder">
    <layout class="com.baeldung.logback.MaskingPatternLayout">
      <maskPattern>\"SSN\"\s*:\s*\"(.*?)\"</maskPattern>
    </layout>
  </encoder>
  <options>
    <dsn>***</dsn>
  </options>
</appender>

I thought this would be similar to setEncoder. But it results in empty events like this:

I guess then I have to figure out how to manually configure Sentry (with Java code?).

@transducer can you please check for processing errors in the Sentry UI? If there's none, you could turn on debug and check the JSON that gets sent to Sentry.

@adinauer I think something else went wrong in my configuration; locally I missed passing a pattern in the XML, thus the whole of Logback was not properly instantiated and onload also an ERROR was visible from Logback ("PatternLayout("null") - Empty or null pattern"). With proper configuration, it does work!

Thanks a lot for the help both.

@adinauer it seems breadcrumbs and messages are encoded, but exception messages are not. Any ideas how exception messages could also be using the encoder? In our use case of scrubbing sensitive information, sometimes sensitive information (like an email address or a database spec) ends up in the message, we want to scrub that.

@transducer can you please specify where exactly you see the message you're referring to? Is this on Sentry UI or on the JSON you can see in debug log?

I'm assuming the problem is this line:

Just trying to confirm before digging deeper. As a workaround for now you should be able to filter data from errors in beforeSend.

@adinauer thanks for your reply. It is in the Sentry UI when logging an uncaught exception. It shows the exception message which is not encoded above the stacktrace. Other log messages and breadcrumbs are encoded.

@transducer thanks. Can you confirm the workaround does fix your problem?

@transducer we've created #2973 to track. We want to discuss this a bit more internally before implementing. Will update there.