Skip to content

Commit adf084c

Browse files
committed
return profiler and converter from options instead of a instance of noop
1 parent 1bb7e19 commit adf084c

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

sentry-spring/src/main/java/io/sentry/spring/SentryProfilerConfiguration.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@
33
import com.jakewharton.nopen.annotation.Open;
44
import io.sentry.IContinuousProfiler;
55
import io.sentry.IProfileConverter;
6-
import io.sentry.NoOpContinuousProfiler;
7-
import io.sentry.NoOpProfileConverter;
86
import io.sentry.Sentry;
97
import io.sentry.SentryOptions;
108
import io.sentry.util.InitUtil;
@@ -26,25 +24,23 @@ public class SentryProfilerConfiguration {
2624
@ConditionalOnMissingBean(name = "sentryOpenTelemetryProfilerConfiguration")
2725
public IContinuousProfiler sentryOpenTelemetryProfilerConfiguration() {
2826
SentryOptions options = Sentry.getGlobalScope().getOptions();
29-
IContinuousProfiler profiler = NoOpContinuousProfiler.getInstance();
3027

3128
if (Sentry.isEnabled()) {
3229
return InitUtil.initializeProfiler(options);
3330
} else {
34-
return profiler;
31+
return options.getContinuousProfiler();
3532
}
3633
}
3734

3835
@Bean
3936
@ConditionalOnMissingBean(name = "sentryOpenTelemetryProfilerConverterConfiguration")
4037
public IProfileConverter sentryOpenTelemetryProfilerConverterConfiguration() {
4138
SentryOptions options = Sentry.getGlobalScope().getOptions();
42-
IProfileConverter converter = NoOpProfileConverter.getInstance();
4339

4440
if (Sentry.isEnabled()) {
4541
return InitUtil.initializeProfileConverter(options);
4642
} else {
47-
return converter;
43+
return options.getProfilerConverter();
4844
}
4945
}
5046
}

0 commit comments

Comments
 (0)