httpi 4.0.4 → 4.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fe1562efbf5ab9aceadc7635826a4cf7317b83d5847c413bdce3462e23728f50
-  data.tar.gz: 2149ada8b2ec173323c1dd248a57d7666bdb9b6a1e0ae83c25e7f96f4c91539b
+  metadata.gz: d6d915a51674973b7ce1d5d435037f117c0c745223e72f992b21b4157c9abdf5
+  data.tar.gz: a4878abbbed22a7dbd1d4f02717066b0a088de74fdf935d0cc24d26ee982e837
 SHA512:
-  metadata.gz: '0964bc62dd38fcd02d9c2c5ae9b07031b7719ccdb3ff831dc6d9f563a1fd0bd085d55e741cd30e063d64b6e3c548d347f8ec9ede647790ed54c4c68d8d50d222'
-  data.tar.gz: 61d4a274d5a330c886ccce1f773ee41567f9d09b216f1eca513ab7646f484e708295548732670a0025726713294c09aa704630c840971a57f79d34db653b97c5
+  metadata.gz: 1709457a67aa799d46d6c7d8afb597b194f21311822d3531f91274961bee70d4a832103dbd6089569da4145d88e163642415d2d1535ae55c71c1ce29232e9115
+  data.tar.gz: e2d82c42ceb35d2967130068e82697e6966c49612367c9930c0ec48e88d59d9150e12dd4e539964526574ec81697dc6b3984bc0c087f6fb937e72aa43813d39c

data/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 * Add your changelog entry here
 
+### 4.0.6 (2026-10-29)
+
+* Republish the 4.0.5 changes as 4.0.6. The 4.0.5 gem never reached RubyGems, as its push workflow failed before the push step. The 4.0.5 tag is protected, so a new version carries the same gem code.
+
+### 4.0.5 (2026-10-29)
+
+* Add `logger` as a direct dependency, so the gem works on Ruby 4.0, which no longer ships `logger` (#276)
+* Fix the Rack adapter to send Content-Type and Content-Length without the `HTTP_` prefix, as the Rack SPEC requires
+
 ### 4.0.4 (2024-07-14)
 
 * gemspec: Exclude spec files from published gem by @ChristopherBert in https://github.com/savonrb/httpi/pull/252

data/README.md

@@ -1,10 +1,10 @@
 # HTTPI
 
-A common interface for Ruby's HTTP libraries. This project is now in maintenance mode. For new projects, we recommend [faraday](https://github.com/lostisland/faraday).
+A common interface for Ruby's HTTP libraries. This project is now in [maintenance mode](https://github.com/savonrb/httpi/issues/238): it receives bug fixes, security updates, and ecosystem-compatibility changes, but no new features. For new projects, we recommend [faraday](https://github.com/lostisland/faraday).
 
 [Documentation](https://www.rubydoc.info/gems/httpi)
 
-[![Development](https://github.com/savonrb/httpi/actions/workflows/development.yml/badge.svg)](https://github.com/savonrb/httpi/actions/workflows/development.yml)
+[![CI](https://github.com/savonrb/httpi/actions/workflows/ci.yml/badge.svg)](https://github.com/savonrb/httpi/actions/workflows/ci.yml)
 
 ## Installation
 

data/lib/httpi/adapter/base.rb

@@ -2,13 +2,11 @@ require "httpi/adapter"
 
 module HTTPI
   module Adapter
-
     # HTTPI::Adapter::Base
     #
     # Allows you to build your own adapter by implementing all public instance methods.
     # Register your adapter by calling the base class' .register method.
     class Base
-
       # Registers an adapter.
       def self.register(name, options = {})
         deps = options.fetch(:deps, [])
@@ -28,7 +26,6 @@ module HTTPI
       def request(method)
         raise NotImplementedError, "Adapters need to implement a #request method"
       end
-
     end
   end
 end

data/lib/httpi/adapter/curb.rb

@@ -3,14 +3,12 @@ require "httpi/response"
 
 module HTTPI
   module Adapter
-
     # = HTTPI::Adapter::Curb
     #
     # Adapter for the Curb client.
     # http://rubygems.org/gems/curb
     class Curb < Base
-
-      register :curb, :deps => %w(curb)
+      register :curb, deps: %w[curb]
 
       def initialize(request)
         @request = request
@@ -98,7 +96,7 @@ module HTTPI
         # The curl man page (http://curl.haxx.se/docs/manpage.html) says that
         # you have to specify a fake username when using Negotiate auth, and
         # they use ':' in their example.
-        @client.username = ':'
+        @client.username = ":"
       end
 
       def setup_ssl_auth
@@ -116,21 +114,21 @@ module HTTPI
           @client.cert_key = ssl.cert_key_file
           @client.cert = ssl.cert_file
           @client.certpassword = ssl.cert_key_password
-          @client.set(:ssl_cipher_list, ssl.ciphers.join(':')) if ssl.ciphers
+          @client.set(:ssl_cipher_list, ssl.ciphers.join(":")) if ssl.ciphers
 
           @client.ssl_verify_peer = ssl.verify_mode == :peer
         end
 
         @client.ssl_version = case ssl.ssl_version
-          when :TLSv1_2 then 1
-          when :TLSv1_1 then 1
-          when :TLSv1   then 1
-          when :SSLv2   then 2
-          when :SSLv23  then 2
-          when :SSLv3   then 3
+        when :TLSv1_2 then 1
+        when :TLSv1_1 then 1
+        when :TLSv1 then 1
+        when :SSLv2 then 2
+        when :SSLv23 then 2
+        when :SSLv3 then 3
         end
         if ssl.min_version || ssl.max_version
-          raise NotSupportedError, 'Curb adapter does not support #min_version or #max_version. Please, use #ssl_version instead.'
+          raise NotSupportedError, "Curb adapter does not support #min_version or #max_version. Please, use #ssl_version instead."
         end
       end
 
@@ -145,15 +143,15 @@ module HTTPI
         status, headers = nil, {}
         return [status, headers] unless header_string
 
-        header_string.split(/\r\n/).each do |header|
-          if header =~ %r|^HTTP/1.[01] \d\d\d (.*)|
+        header_string.split("\r\n").each do |header|
+          if header =~ %r{^HTTP/1.[01] \d\d\d (.*)}
             status = $1
           else
-            parts = header.split(':', 2)
+            parts = header.split(":", 2)
             unless parts.empty?
-              parts[1].strip! unless parts[1].nil?
+              parts[1]&.strip!
               if headers.has_key?(parts[0])
-                headers[parts[0]] = [headers[parts[0]]] unless headers[parts[0]].kind_of? Array
+                headers[parts[0]] = [headers[parts[0]]] unless headers[parts[0]].is_a? Array
                 headers[parts[0]] << parts[1]
               else
                 headers[parts[0]] = parts[1]
@@ -164,7 +162,6 @@ module HTTPI
 
         [status, headers]
       end
-
     end
   end
 end

data/lib/httpi/adapter/em_http.rb

@@ -3,7 +3,6 @@ require "httpi/response"
 
 module HTTPI
   module Adapter
-
     # An HTTPI adapter for `EventMachine::HttpRequest`. Due to limitations of
     # the em-httprequest library, not all features are supported. In particular,
     #
@@ -28,8 +27,7 @@ module HTTPI
     #
     # are supported by em-httprequest but not HTTPI.
     class EmHttpRequest < Base
-
-      register :em_http, :deps => %w(em-synchrony em-synchrony/em-http em-http)
+      register :em_http, deps: %w[em-synchrony em-synchrony/em-http em-http]
 
       def initialize(request)
         @request = request
@@ -82,17 +80,17 @@ module HTTPI
 
       def client_options
         {
-          :query => @request.url.query,
-          :head  => @request.headers.to_hash,
-          :body  => @request.body
+          query: @request.url.query,
+          head: @request.headers.to_hash,
+          body: @request.body
         }
       end
 
       def proxy_options
         {
-          :host          => @request.proxy.host,
-          :port          => @request.proxy.port,
-          :authorization => [@request.proxy.user, @request.proxy.password]
+          host: @request.proxy.host,
+          port: @request.proxy.port,
+          authorization: [@request.proxy.user, @request.proxy.password]
         }
       end
 
@@ -134,6 +132,5 @@ module HTTPI
         result
       end
     end
-
   end
 end

data/lib/httpi/adapter/excon.rb

@@ -3,18 +3,16 @@ require "httpi/response"
 
 module HTTPI
   module Adapter
-
     # = HTTPI::Adapter::Excon
     #
     # Adapter for the Excon client.
     # https://github.com/geemus/excon
     class Excon < Base
-
-      register :excon, :deps => %w(excon)
+      register :excon, deps: %w[excon]
 
       def initialize(request)
         @request = request
-        @client  = ::Excon::Connection.new client_opts
+        @client = ::Excon::Connection.new client_opts
       end
 
       attr_reader :client
@@ -36,18 +34,18 @@ module HTTPI
       private
 
       def client_opts
-        url  = @request.url
-        ssl  = @request.auth.ssl
+        url = @request.url
+        ssl = @request.auth.ssl
 
         opts = {
-          :host => url.host,
-          :hostname => url.hostname,
-          :path => url.path,
-          :port => url.port,
-          :query => url.query,
-          :scheme => url.scheme,
-          :headers => @request.headers,
-          :body => @request.body
+          host: url.host,
+          hostname: url.hostname,
+          path: url.path,
+          port: url.port,
+          query: url.query,
+          scheme: url.scheme,
+          headers: @request.headers,
+          body: @request.body
         }
 
         if @request.auth.digest?
@@ -58,16 +56,16 @@ module HTTPI
 
         opts[:user], opts[:password] = *@request.auth.credentials if @request.auth.basic?
         opts[:connect_timeout] = @request.open_timeout if @request.open_timeout
-        opts[:read_timeout]    = @request.read_timeout if @request.read_timeout
-        opts[:write_timeout]   = @request.write_timeout if @request.write_timeout
-        opts[:response_block]  = @request.on_body if @request.on_body
-        opts[:proxy]           = @request.proxy if @request.proxy
+        opts[:read_timeout] = @request.read_timeout if @request.read_timeout
+        opts[:write_timeout] = @request.write_timeout if @request.write_timeout
+        opts[:response_block] = @request.on_body if @request.on_body
+        opts[:proxy] = @request.proxy if @request.proxy
 
         case ssl.verify_mode
         when :peer
           opts[:ssl_verify_peer] = true
           opts[:ssl_ca_file] = ssl.ca_cert_file if ssl.ca_cert_file
-          opts[:certificate] = ssl.cert.to_pem     if ssl.cert
+          opts[:certificate] = ssl.cert.to_pem if ssl.cert
           opts[:private_key] = ssl.cert_key.to_pem if ssl.cert_key
         when :none
           opts[:ssl_verify_peer] = false
@@ -88,7 +86,6 @@ module HTTPI
         end
         Response.new response.status, headers, response.body
       end
-
     end
   end
 end

data/lib/httpi/adapter/http.rb

@@ -3,14 +3,12 @@ require "httpi/response"
 
 module HTTPI
   module Adapter
-
     # = HTTPI::Adapter::HTTP
     #
     # Adapter for the http.rb client.
     # https://github.com/httprb/http.rb
     class HTTP < Base
-
-      register :http, :deps => %w(http)
+      register :http, deps: %w[http]
 
       def initialize(request)
         if request.auth.digest?
@@ -33,7 +31,7 @@ module HTTPI
           raise NotSupportedError, "http.rb does not support custom HTTP methods"
         end
         response = begin
-          @client.send(method, @request.url, :body => @request.body)
+          @client.send(method, @request.url, body: @request.body)
         rescue OpenSSL::SSL::SSLError
           raise SSLError
         end
@@ -49,30 +47,30 @@ module HTTPI
 
           context.options = OpenSSL::SSL::SSLContext::DEFAULT_PARAMS[:options]
 
-          if @request.auth.ssl.ca_cert_file != nil
+          if !@request.auth.ssl.ca_cert_file.nil?
             context.ca_file = @request.auth.ssl.ca_cert_file
           else
             context.cert_store = OpenSSL::SSL::SSLContext::DEFAULT_CERT_STORE
           end
 
-          context.cert        = @request.auth.ssl.cert
-          context.key         = @request.auth.ssl.cert_key
-          context.ssl_version = @request.auth.ssl.ssl_version if @request.auth.ssl.ssl_version != nil
-          context.min_version = @request.auth.ssl.min_version if @request.auth.ssl.min_version != nil
-          context.max_version = @request.auth.ssl.max_version if @request.auth.ssl.max_version != nil
+          context.cert = @request.auth.ssl.cert
+          context.key = @request.auth.ssl.cert_key
+          context.ssl_version = @request.auth.ssl.ssl_version if !@request.auth.ssl.ssl_version.nil?
+          context.min_version = @request.auth.ssl.min_version if !@request.auth.ssl.min_version.nil?
+          context.max_version = @request.auth.ssl.max_version if !@request.auth.ssl.max_version.nil?
           context.verify_mode = @request.auth.ssl.openssl_verify_mode
-          context.ciphers     = @request.auth.ssl.ciphers if @request.auth.ssl.ciphers
+          context.ciphers = @request.auth.ssl.ciphers if @request.auth.ssl.ciphers
 
-          client = ::HTTP::Client.new(:ssl_context => context)
+          client = ::HTTP::Client.new(ssl_context: context)
         else
           client = ::HTTP
         end
 
         if @request.auth.basic?
-          client = client.basic_auth(:user => @request.auth.credentials[0], :pass => @request.auth.credentials[1])
+          client = client.basic_auth(user: @request.auth.credentials[0], pass: @request.auth.credentials[1])
         end
 
-        if @request.proxy != nil
+        if !@request.proxy.nil?
           client = client.via(@request.proxy.host, @request.proxy.port, @request.proxy.user, @request.proxy.password)
         end
 

data/lib/httpi/adapter/httpclient.rb

@@ -3,14 +3,12 @@ require "httpi/response"
 
 module HTTPI
   module Adapter
-
     # = HTTPI::Adapter::HTTPClient
     #
     # Adapter for the HTTPClient client.
     # http://rubygems.org/gems/httpclient
     class HTTPClient < Base
-
-      register :httpclient, :deps => %w(httpclient)
+      register :httpclient, deps: %w[httpclient]
 
       def initialize(request)
         @request = request
@@ -80,22 +78,21 @@ module HTTPI
 
         @client.ssl_config.ssl_version = ssl.ssl_version.to_s if ssl.ssl_version
         if ssl.min_version || ssl.max_version
-          raise NotSupportedError, 'Httpclient adapter does not support #min_version or #max_version. Please, use #ssl_version instead'
+          raise NotSupportedError, "Httpclient adapter does not support #min_version or #max_version. Please, use #ssl_version instead"
         end
       end
 
       def respond_with(response)
         headers = {}
         response.header.all.each do |(header, value)|
-          if headers.key?(header)
-            headers[header] = Array(headers[header]) << value
+          headers[header] = if headers.key?(header)
+            Array(headers[header]) << value
           else
-            headers[header] = value
+            value
           end
         end
         Response.new response.code, headers, response.content
       end
-
     end
   end
 end

data/lib/httpi/adapter/net_http.rb

@@ -2,19 +2,17 @@ require "uri"
 
 require "httpi/adapter/base"
 require "httpi/response"
-require 'kconv'
-require 'socket'
+require "kconv"
+require "socket"
 
 module HTTPI
   module Adapter
-
     # = HTTPI::Adapter::NetHTTP
     #
     # Adapter for the Net::HTTP client.
     # http://ruby-doc.org/stdlib/libdoc/net/http/rdoc/
     class NetHTTP < Base
-
-      register :net_http, :deps => %w(net/https)
+      register :net_http, deps: %w[net/https]
       def initialize(request)
         check_net_ntlm_version! if request.auth.ntlm?
         @request = request
@@ -28,14 +26,14 @@ module HTTPI
       def request(method)
         # Determine if Net::HTTP supports the method using reflection
         unless Net::HTTP.const_defined?(:"#{method.to_s.capitalize}") &&
-            Net::HTTP.const_get(:"#{method.to_s.capitalize}").class == Class
+            Net::HTTP.const_get(:"#{method.to_s.capitalize}").instance_of?(Class)
 
-          raise NotSupportedError, "Net::HTTP does not support "\
+          raise NotSupportedError, "Net::HTTP does not support " \
             "#{method.to_s.upcase}"
         end
         do_request(method) do |http, http_request|
           http_request.body = @request.body
-          if @request.on_body then
+          if @request.on_body
             perform(http, http_request) do |res|
               res.read_body do |seg|
                 @request.on_body.call(seg)
@@ -53,19 +51,18 @@ module HTTPI
       end
 
       private
+
       def ntlm_version
         Net::NTLM::VERSION::STRING
       end
 
       def check_net_ntlm_version!
-        begin
-          require 'net/ntlm'
-          require 'net/ntlm/version' unless Net::NTLM.const_defined?(:VERSION, false)
-          unless ntlm_version >= '0.3.2'
-            raise ArgumentError, 'Invalid version of rubyntlm. Please use v0.3.2+.'
-          end
-        rescue LoadError
+        require "net/ntlm"
+        require "net/ntlm/version" unless Net::NTLM.const_defined?(:VERSION, false)
+        unless ntlm_version >= "0.3.2"
+          raise ArgumentError, "Invalid version of rubyntlm. Please use v0.3.2+."
         end
+      rescue LoadError
       end
 
       def perform(http, http_request, &block)
@@ -74,10 +71,10 @@ module HTTPI
 
       def create_client
         proxy_url = @request.proxy || URI("")
-        if URI(proxy_url).scheme == 'socks'
-          proxy =Net::HTTP.SOCKSProxy(proxy_url.host, proxy_url.port)
+        proxy = if URI(proxy_url).scheme == "socks"
+          Net::HTTP.SOCKSProxy(proxy_url.host, proxy_url.port)
         else
-          proxy = Net::HTTP::Proxy(proxy_url.host, proxy_url.port, proxy_url.user, proxy_url.password)
+          Net::HTTP::Proxy(proxy_url.host, proxy_url.port, proxy_url.user, proxy_url.password)
         end
         proxy.new(@request.url.host, @request.url.port)
       end
@@ -98,24 +95,24 @@ module HTTPI
 
       def negotiate_ntlm_auth(http, &requester)
         unless Net.const_defined?(:NTLM)
-          raise NotSupportedError, 'Net::NTLM is not available. Install via gem install rubyntlm.'
+          raise NotSupportedError, "Net::NTLM is not available. Install via gem install rubyntlm."
         end
 
         # first figure out if we should use NTLM or Negotiate
         nego_auth_response = respond_with(requester.call(http, request_client(:head)))
-        if nego_auth_response.headers['www-authenticate'] && nego_auth_response.headers['www-authenticate'].include?('Negotiate')
-          auth_method = 'Negotiate'
-        elsif nego_auth_response.headers['www-authenticate'] && nego_auth_response.headers['www-authenticate'].include?('NTLM')
-          auth_method = 'NTLM'
+        if nego_auth_response.headers["www-authenticate"]&.include?("Negotiate")
+          auth_method = "Negotiate"
+        elsif nego_auth_response.headers["www-authenticate"]&.include?("NTLM")
+          auth_method = "NTLM"
         else
-          auth_method = 'NTLM'
-          HTTPI.logger.debug 'Server does not support NTLM/Negotiate. Trying NTLM anyway'
+          auth_method = "NTLM"
+          HTTPI.logger.debug "Server does not support NTLM/Negotiate. Trying NTLM anyway"
         end
 
         # initiate a request is to authenticate (exchange secret and auth) using the method determined above...
         ntlm_message_type1 = Net::NTLM::Message::Type1.new
-        %w(workstation domain).each do |a|
-          ntlm_message_type1.send("#{a}=",'')
+        %w[workstation domain].each do |a|
+          ntlm_message_type1.send("#{a}=", "")
           ntlm_message_type1.enable(a.to_sym)
         end
 
@@ -134,14 +131,14 @@ module HTTPI
           message_builder[:password] = @request.auth.ntlm[1]
 
           # we need to provide a domain in the packet if an only if it was provided by the user in the auth request
-          if @request.auth.ntlm[2]
-            message_builder[:domain] = @request.auth.ntlm[2].upcase
+          message_builder[:domain] = if @request.auth.ntlm[2]
+            @request.auth.ntlm[2].upcase
           else
-            message_builder[:domain] = ''
+            ""
           end
 
-          ntlm_response = ntlm_message.response(message_builder ,
-                                                 {:ntlmv2 => true})
+          ntlm_response = ntlm_message.response(message_builder,
+            {ntlmv2: true})
           # Finally add header of Authorization
           @request.headers["Authorization"] = "#{auth_method} #{ntlm_response.encode64}"
         end
@@ -212,10 +209,9 @@ module HTTPI
         headers.each do |key, value|
           headers[key] = value[0] if value.size <= 1
         end
-        body = (response.body.kind_of?(Net::ReadAdapter) ? "" : response.body)
+        body = (response.body.is_a?(Net::ReadAdapter) ? "" : response.body)
         Response.new response.code, headers, body
       end
-
     end
   end
 end

data/lib/httpi/adapter/net_http_persistent.rb

@@ -1,18 +1,16 @@
 module HTTPI
   module Adapter
-
     # = HTTPI::Adapter::NetHTTPPersistent
     #
     # Adapter for the Net::HTTP::Persistent client.
     # http://docs.seattlerb.org/net-http-persistent/Net/HTTP/Persistent.html
     class NetHTTPPersistent < NetHTTP
-
-      register :net_http_persistent, :deps => %w(net/http/persistent)
+      register :net_http_persistent, deps: %w[net/http/persistent]
 
       private
 
       def create_client
-        if Gem::Version.new(Net::HTTP::Persistent::VERSION) >= Gem::Version.new('3.0.0')
+        if Gem::Version.new(Net::HTTP::Persistent::VERSION) >= Gem::Version.new("3.0.0")
           Net::HTTP::Persistent.new name: thread_key
         else
           Net::HTTP::Persistent.new thread_key
@@ -40,7 +38,7 @@ module HTTPI
       end
 
       def thread_key
-        @request.url.host.split(/\W/).reject{|p|p == ""}.join('-')
+        @request.url.host.split(/\W/).reject { |p| p == "" }.join("-")
       end
     end
   end

data/lib/httpi/adapter/rack.rb

@@ -1,10 +1,9 @@
-require 'base64'
-require 'httpi/adapter/base'
-require 'httpi/response'
+require "base64"
+require "httpi/adapter/base"
+require "httpi/response"
 
 module HTTPI
   module Adapter
-
     # = HTTPI::Adapter::Rack
     #
     # Adapter for Rack::MockRequest.
@@ -16,7 +15,7 @@ module HTTPI
     #   HTTPI::Adapter::Rack.mount 'application', RackApplication
     #   HTTPI.get("http://application/path", :rack)
     class Rack < Base
-      register :rack, :deps => %w(rack/mock)
+      register :rack, deps: %w[rack/mock]
 
       attr_reader :client
 
@@ -29,27 +28,26 @@ module HTTPI
       # Attaches Rack endpoint at specified host.
       # Endpoint will be acessible at {http://host/ http://host/} url.
       def self.mount(host, application)
-        self.mounted_apps[host] = application
+        mounted_apps[host] = application
       end
 
       # Removes Rack endpoint.
       def self.unmount(host)
-        self.mounted_apps.delete(host)
+        mounted_apps.delete(host)
       end
 
       def initialize(request)
         @app = self.class.mounted_apps[request.url.host]
 
-
         if @app.nil?
-          message  = "Application '#{request.url.host}' not mounted: ";
+          message = "Application '#{request.url.host}' not mounted: "
           message += "use `HTTPI::Adapter::Rack.mount('#{request.url.host}', RackApplicationClass)`"
 
           raise message
         end
 
         @request = request
-        @client  = ::Rack::MockRequest.new(@app)
+        @client = ::Rack::MockRequest.new(@app)
       end
 
       # Executes arbitrary HTTP requests.
@@ -68,9 +66,9 @@ module HTTPI
 
         if @request.auth.http?
           if @request.auth.basic?
-            basic_auth = @request.auth.basic.join(':')
-            encoded = Base64.encode64(basic_auth).gsub('\n', '')
-            @request.headers['Authorization'] = "Basic #{encoded}"
+            basic_auth = @request.auth.basic.join(":")
+            encoded = Base64.encode64(basic_auth).gsub('\n', "")
+            @request.headers["Authorization"] = "Basic #{encoded}"
           else
             raise NotSupportedError, "Rack adapter does not support HTTP #{@request.auth.type} auth"
           end
@@ -86,14 +84,24 @@ module HTTPI
 
         env = {}
         @request.headers.each do |header, value|
-          env["HTTP_#{header.gsub('-', '_').upcase}"] = value
+          # The headers CONTENT_TYPE and CONTENT_LENGTH should not have "HTTP_"
+          # added as a prefix.  For all other headers, add the prefix "HTTP_"
+          # https://github.com/rack/rack/blob/main/SPEC.rdoc#label-HTTP_+Headers
+          normalized = header.tr("-", "_").upcase
+          key = if normalized == "CONTENT_TYPE" || normalized == "CONTENT_LENGTH"
+            normalized
+          else
+            "HTTP_#{normalized}"
+          end
+
+          env[key] = value
         end
 
         response = @client.request(method.to_s.upcase, @request.url.to_s,
-              { :fatal => true, :input => @request.body.to_s }.merge(env))
+          {fatal: true, input: @request.body.to_s}.merge(env))
 
         Response.new(response.status, response.headers, response.body)
       end
     end
   end
-end
+end