Skip to content

Commit cca089a

Browse files
committed
WIP - address stack errors
1 parent 8d39305 commit cca089a

File tree

1 file changed

+11
-9
lines changed

1 file changed

+11
-9
lines changed

sentry-ruby/lib/sentry/std_lib_logger.rb

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,24 +16,26 @@ def add(severity, message = nil, progname = nil, &block)
1616
result = super
1717

1818
return result unless Sentry.initialized? && Sentry.get_current_hub
19+
return result unless Sentry.configuration.enabled_patches.include?(:logger) && Sentry.configuration.enable_logs
1920

20-
# exclude sentry SDK logs -- to prevent recursive log action,
21-
# do not process internal logs again
22-
if message.nil? && progname != Sentry::Logger::PROGNAME
21+
return result if progname == Sentry::Logger::PROGNAME
2322

24-
# handle different nature of Ruby Logger class:
25-
# inspo from Sentry::Breadcrumb::SentryLogger
23+
if message.nil?
2624
if block_given?
2725
message = yield
2826
else
2927
message = progname
3028
end
29+
end
3130

32-
message = message.to_s.strip
31+
return result if message.nil? || message.to_s.strip.empty?
3332

34-
if !message.nil? && message != Sentry::Logger::PROGNAME && method = SEVERITY_MAP[severity]
35-
Sentry.logger.send(method, message)
36-
end
33+
message = message.to_s.strip
34+
35+
return result if message == Sentry::Logger::PROGNAME
36+
37+
if method = SEVERITY_MAP[severity]
38+
Sentry.logger.send(method, message)
3739
end
3840

3941
result

0 commit comments

Comments
 (0)