fix: fixes potential issue if machine hostname cannot be determined

postalserver · Feb 24, 2024 · 0fcf778 · 0fcf778
1 parent 2e7b36c
commit 0fcf778
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/app/util/health_server.rb b/app/util/health_server.rb
@@ -26,7 +26,7 @@ def call(env)
  private
 
  def root
- [200, { "Content-Type" => "text/plain" }, ["#{@name} (pid: #{Process.pid}, host: #{Socket.gethostname})"]]
+ [200, { "Content-Type" => "text/plain" }, ["#{@name} (pid: #{Process.pid}, host: #{hostname})"]]
  end
 
  def ok
@@ -43,6 +43,12 @@ def metrics
  [200, { "Content-Type" => "text/plain" }, [body]]
  end
 
+ def hostname
+ Socket.gethostname
+ rescue StandardError
+ "unknown-hostname"
+ end
+
  class << self
 
  def run(default_port: 9090, **options)