Fixes #38120 - Support Sinatra 4 and Rack 3

Author: lhellebr

lib/launcher.rb

@@ -1,4 +1,5 @@
 require 'openssl'
+require 'rackup/handler/webrick'
 require 'proxy/log'
 require 'proxy/settings'
 require 'proxy/signal_handler'
@@ -191,7 +192,7 @@ def webrick_server(app, addresses, port)
       rescue ::OpenSSL::SSL::SSLError => e
         raise "Invalid tls_ciphers value '#{app[:SSLCiphers]}': #{e.message}"
       end
-      server.mount "/", Rack::Handler::WEBrick, app[:app]
+      server.mount "/", Rackup::Handler::WEBrick, app[:app]
 
       # WEBrick 1.9.x does not support :SSLMinVersion in its config hash, so we
       # apply min_version= directly on the SSL context after WEBrick creates it.

lib/proxy/hsts_middleware.rb

@@ -10,8 +10,8 @@ def initialize(app)
 
     def call(env)
       status, headers, body = @app.call(env)
-      if env['HTTPS'] == 'on' && !headers.include?('Strict-Transport-Security')
-        headers['Strict-Transport-Security'] = 'max-age=31536000'
+      if env['HTTPS'] == 'on' && !headers.include?('strict-transport-security')
+        headers['strict-transport-security'] = 'max-age=31536000'
       end
       [status, headers, body]
     end

modules/registration/registration_api.rb

@@ -1,6 +1,11 @@
 require 'registration/proxy_request'
 
 class Proxy::Registration::Api < ::Sinatra::Base
+  # Needed so `logger` resolves to Proxy::LogBuffer::Decorator (which implements
+  # #exception, used in the rescue blocks below) instead of Sinatra's own null
+  # logger, which is a plain ::Logger with no #exception method as of Sinatra 4.
+  helpers ::Proxy::Helpers
+
   # Cache for the global registration script (GET /register).
   #
   # The script is identical for all hosts sharing the same registration

smart_proxy.gemspec

@@ -17,10 +17,11 @@ Gem::Specification.new do |s|
   s.add_dependency 'json'
   s.add_dependency 'logging'
   s.add_dependency 'ostruct'
-  s.add_dependency 'rack', '>= 1.3'
+  s.add_dependency 'rack', '>= 3.0', '< 4'
+  s.add_dependency 'rackup', '~> 2.0'
   s.add_dependency 'rexml', '~> 3.2'
   s.add_dependency 'sd_notify', '~> 0.1'
-  s.add_dependency 'sinatra', '~> 2.0'
+  s.add_dependency 'sinatra', '~> 4.0'
   s.add_dependency 'webrick', '~> 1.0'
   s.description = <<~EOF
     Foreman Proxy is used via The Foreman Project, it allows Foreman to manage