Sentry Logback with Spring Boot does not pickup configuration from Spring

[rharrahuzurv]

Integration

sentry

Java Version

24

Version

8.15.0

Steps to Reproduce

Use a Spring Boot 3.4.5 application with Sentry 8.15.0 and enable Sentry logging:

sentry:
  dsn: mydsngoeshere
  send-default-pii: true
  environment: ${ENVIRONMENT_NAME}
  logs:
    enabled: true
  logging:
    minimum-level: info
    minimum-event-level: info
    minimum-breadcrumb-level: debug
    enabled: true

Expected Result

Logs should appear in Sentry Logs.

Actual Result

Logs do not appear in Sentry logs.

As of this line (https://github.com/getsentry/sentry-java/blob/main/sentry-spring-boot-jakarta/src/main/java/io/sentry/spring/boot/jakarta/SentryLogbackInitializer.java#L58), SentryOptions$Logs is correctly detecting from the Spring properties that logs are enabled:

As of this line (https://github.com/getsentry/sentry-java/blob/8.15.0/sentry-logback/src/main/java/io/sentry/logback/SentryAppender.java#L71), SentryOptions$Logs is not using the properties, which is to be expected because these are external configurations:

Once it reaches and invokes the method on this line (https://github.com/getsentry/sentry-java/blob/8.15.0/sentry/src/main/java/io/sentry/Sentry.java#L291) in Sentry.init, it should pick up the external configuration from Spring but does not. It resolves values from environment variables / system properties / classpath but never receives either the DSN or that logs are enabled configured in the Spring properties, resulting in logs never being sent:

[adinauer]

Thanks for opening this issue @rharrahuzurv we'll investigate. The SDK should re-initialize once Spring Boot has been loaded and then apply your application.yml.

So startup logs are not captured but anything after the Spring Boot Sentry.init should.
I presume you don't see any logs even after your applications has fully started, correct?

[adinauer]

Ah I think I found the issue. Will probably PR a fix soon.

[adinauer]

@rharrahuzurv https://github.com/getsentry/sentry-java/releases/tag/8.15.1 has just been released and should fix the problem.

[rharrahuzurv]

@adinauer I'm still seeing the issue but am getting a new logged warning, at least. Using 8.15.1:

from console after enabling logback debugging:

13:23:16,511 |-WARN in io.sentry.logback.SentryAppender[SENTRY_APPENDER] - Failed to init Sentry during appender initialization: DSN is required. Use empty string or set enabled to false in SentryOptions to disable SDK.

[rharrahuzurv]

The above is a false positive, it's working now. Sorry for the confusion!