Skip to content

Commit 34510cb

Browse files
ndbroadbentclaude
andcommitted
fix puma_server? - check $PROGRAM_NAME, not defined?(Puma::Server)
Just checking defined?(::Puma::Server) is unreliable because Puma might be installed but not running. Check $PROGRAM_NAME.include?("puma") to verify we're actually running as a puma process. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 509d2cf commit 34510cb

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

lib/log_struct/concerns/configuration.rb

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ def console_process?
132132
sig { returns(T::Boolean) }
133133
def server_process?
134134
return true if logstruct_server_mode?
135-
return true if defined?(::Puma::Server)
135+
return true if puma_server?
136136
return true if defined?(::Unicorn::HttpServer)
137137
return true if defined?(::Thin::Server)
138138
return true if defined?(::Falcon::Server)
@@ -142,6 +142,13 @@ def server_process?
142142
false
143143
end
144144

145+
sig { returns(T::Boolean) }
146+
def puma_server?
147+
# Just checking defined?(::Puma::Server) is not reliable - Puma might be installed
148+
# but not running. Check $PROGRAM_NAME to verify we're actually running puma.
149+
$PROGRAM_NAME.include?("puma")
150+
end
151+
145152
sig { returns(T::Boolean) }
146153
def sidekiq_server?
147154
!!(defined?(::Sidekiq) && ::Sidekiq.respond_to?(:server?) && ::Sidekiq.server?)

0 commit comments

Comments
 (0)