Ensure compatibility with --enable-frozen-string-literal

.github/workflows/ci.yml

@@ -25,5 +25,5 @@ jobs:
       - name: Install packages
         run: sudo apt-get install -y ragel socat netcat
 
-      - name: Tests
+      - name: Tests ${{ matrix.rubyopt }}
         run: bundle exec rake

Gemfile.lock

@@ -31,4 +31,4 @@ DEPENDENCIES
   rake-compiler
 
 BUNDLED WITH
-   2.3.22
+   2.3.27

benchmark/cow_benchmark.rb

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 require "net/http"
 
 app_path = File.expand_path('../examples/constant_caches.ru', __dir__)

examples/constant_caches.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative "../lib/pitchfork/mem_info"
 
 module App

examples/echo.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Example application that echoes read data back to the HTTP client.
 # This emulates the old echo protocol people used to run.
 #

examples/hello.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 run lambda { |env|
   /\A100-continue\z/i =~ env['HTTP_EXPECT'] and return [100, {}, []]
   body = "Hello World!\n"

examples/pitchfork.conf.minimal.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Minimal sample configuration file for Pitchfork
 
 # listen 2007 # by default Pitchfork listens on port 8080

examples/pitchfork.conf.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Sample verbose configuration file for Pitchfork
 
 # Use at least one worker per core if you're on a dedicated server,

exe/pitchfork

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # -*- encoding: binary -*-
 require 'pitchfork/launcher'
 require 'optparse'

ext/pitchfork_http/extconf.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 require 'mkmf'
 
 have_const("PR_SET_CHILD_SUBREAPER", "sys/prctl.h")

lib/pitchfork.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 require 'etc'
 require 'stringio'
 require 'raindrops'

lib/pitchfork/app/old_rails/static.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 # :enddoc:
 # This code is based on the original Rails handler in Mongrel
 # Copyright (c) 2005 Zed A. Shaw

lib/pitchfork/children.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 
 module Pitchfork
   # This class keep tracks of the state of all the master children.

lib/pitchfork/configurator.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 require 'logger'
 
 module Pitchfork

lib/pitchfork/const.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 
 module Pitchfork
   module Const # :nodoc:

lib/pitchfork/flock.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'tempfile'
 
 module Pitchfork

lib/pitchfork/http_parser.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 # :enddoc:
 # no stable API here
 
@@ -19,7 +20,7 @@ class HttpParser
       "SERVER_SOFTWARE" => "Pitchfork #{Pitchfork::Const::UNICORN_VERSION}"
     }
 
-    NULL_IO = StringIO.new("")
+    NULL_IO = StringIO.new.binmode
 
     # :stopdoc:
     HTTP_RESPONSE_START = [ 'HTTP'.freeze, '/1.1 '.freeze ]

lib/pitchfork/http_response.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 # :enddoc:
 
 module Pitchfork

lib/pitchfork/http_server.rb

@@ -1,4 +1,6 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
+
 require 'pitchfork/pitchfork_http'
 require 'pitchfork/flock'
 require 'pitchfork/soft_timeout'
@@ -696,7 +698,7 @@ def handle_error(client, e)
 
     def e103_response_write(client, headers)
       rss = @request.response_start_sent
-      buf = rss ? "103 Early Hints\r\n" : "HTTP/1.1 103 Early Hints\r\n"
+      buf = (rss ? "103 Early Hints\r\n" : "HTTP/1.1 103 Early Hints\r\n").b
       headers.each { |key, value| append_header(buf, key, value) }
       buf << (rss ? "\r\nHTTP/1.1 ".freeze : "\r\n".freeze)
       client.write(buf)

lib/pitchfork/launcher.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 
 # :enddoc:
 $stdout.sync = $stderr.sync = true

lib/pitchfork/message.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 
 # :stopdoc:
 module Pitchfork

lib/pitchfork/select_waiter.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Pitchfork
   # fallback for non-Linux and Linux <4.5 systems w/o EPOLLEXCLUSIVE
   class SelectWaiter # :nodoc:

lib/pitchfork/socket_helper.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 # :enddoc:
 require 'socket'
 

lib/pitchfork/stream_input.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 
 module Pitchfork
   # When processing uploads, pitchfork may expose a StreamInput object under
@@ -17,7 +18,7 @@ def initialize(socket, request) # :nodoc:
       @socket = socket
       @parser = request
       @buf = request.buf
-      @rbuf = ''
+      @rbuf = +''
       @bytes_read = 0
       filter_body(@rbuf, @buf) unless @buf.empty?
     end
@@ -41,7 +42,7 @@ def initialize(socket, request) # :nodoc:
     # ios.read(length [, buffer]) will return immediately if there is
     # any data and only block when nothing is available (providing
     # IO#readpartial semantics).
-    def read(length = nil, rv = '')
+    def read(length = nil, rv = ''.b)
       if length
         if length <= @rbuf.size
           length < 0 and raise ArgumentError, "negative length #{length} given"
@@ -79,16 +80,16 @@ def read(length = nil, rv = '')
     # unlike IO#gets.
     def gets(sep = $/)
       if sep.nil?
-        read_all(rv = '')
+        read_all(rv = ''.b)
         return rv.empty? ? nil : rv
       end
       re = /\A(.*?#{Regexp.escape(sep)})/
 
       begin
-        @rbuf.sub!(re, '') and return $1
+        @rbuf.sub!(re, ''.b) and return $1
         return @rbuf.empty? ? nil : @rbuf.slice!(0, @rbuf.size) if eof?
         @socket.readpartial(@@io_chunk_size, @buf) or eof!
-        filter_body(once = '', @buf)
+        filter_body(once = ''.b, @buf)
         @rbuf << once
       end while true
     end

lib/pitchfork/tee_input.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 
 module Pitchfork
   # Acts like tee(1) on an input input to provide a input-like stream
@@ -42,7 +43,7 @@ def initialize(socket, request) # :nodoc:
       @len = request.content_length
       super
       @tmp = @len && @len <= @@client_body_buffer_size ?
-             StringIO.new("") : new_tmpio
+             StringIO.new.binmode : new_tmpio
     end
 
     # :call-seq:
@@ -121,7 +122,7 @@ def rewind
 
     # consumes the stream of the socket
     def consume!
-      junk = ""
+      junk = "".b
       nil while read(@@io_chunk_size, junk)
     end
 

lib/pitchfork/tmpio.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 # :stopdoc:
 require 'tmpdir'
 

lib/pitchfork/worker.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 require 'pitchfork/shared_memory'
 
 module Pitchfork

test/integration/apps/chunked.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class ChunkedBody
   def each(&block)
     ('A'..'Z').each do |char|

test/integration/apps/stream.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class ChunkedBody
   def each(&block)
     ('A'..'Z').each do |char|

test/integration/bin/content-md5-put

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # -*- encoding: binary -*-
 # simple chunked HTTP PUT request generator (and just that),
 # it reads stdin and writes to stdout so socat can write to a

test/integration/bin/sha1sum.rb

@@ -1,5 +1,6 @@
 #!/usr/bin/env ruby
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 # Reads from stdin and outputs the SHA1 hex digest of the input
 
 require 'digest/sha1'

test/integration/bin/unused_listen

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # -*- encoding: binary -*-
 # this is to remain compatible with the unused_port function in the
 # Unicorn test/test_helper.rb file

test/integration/broken-app.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # we do not want Rack::Lint or anything to protect us
 use Rack::ContentLength
 use Rack::ContentType, "text/plain"

test/integration/env.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 use Rack::ContentLength
 use Rack::ContentType, "text/plain"
 run lambda { |env| [ 200, {}, [ env.inspect << "\n" ] ] }

test/integration/fails-rack-lint.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This rack app returns a header with key named "status", which will cause
 # Rack::Lint to throw an exception if it is present.  This
 # is used to check whether Rack::Lint is in the stack or not.

test/integration/fork_unsafe.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 use Rack::ContentLength
 use Rack::ContentType, "text/plain"
 run lambda { |env|

test/integration/heartbeat-timeout.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 use Rack::ContentLength
 headers = { 'content-type' => 'text/plain' }
 run lambda { |env|

test/integration/info.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 use Rack::ContentLength
 use Rack::ContentType, "text/plain"
 run lambda { |env|

test/integration/listener_names.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 use Rack::ContentLength
 use Rack::ContentType, "text/plain"
 run(lambda { |_| [ 200, {}, [ Pitchfork.listener_names.inspect ] ] })

test/integration/pid.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 use Rack::ContentLength
 use Rack::ContentType, "text/plain"
 run lambda { |env| [ 200, {}, [ "#$$\n" ] ] }

test/integration/rack-input-tests.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # SHA1 checksum generator
 require 'digest/sha1'
 use Rack::ContentLength

test/integration/sleep.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 use Rack::ContentLength
 use Rack::ContentType, "text/plain"
 run lambda { |env|

test/integration/t0013.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #\ -E none
 use Rack::ContentLength
 use Rack::ContentType, 'text/plain'

test/integration/t0014.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #\ -E none
 use Rack::ContentLength
 use Rack::ContentType, 'text/plain'

test/integration/t0116.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #\ -E none
 use Rack::ContentLength
 use Rack::ContentType, 'text/plain'

test/integration/t0301.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #\-N --debug
 run(lambda do |env|
   case env['PATH_INFO']

test/integration/t2000.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 use Rack::ContentLength
 use Rack::ContentType, "text/plain"
 run lambda { |env|

test/integration/test_boot.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'integration_test_helper'
 
 class TestBoot < Pitchfork::IntegrationTest

test/integration/test_broken_app.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'integration_test_helper'
 
 class BrokenAppTest < Pitchfork::IntegrationTest

test/integration/test_configuration.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'integration_test_helper'
 
 class ConfigurationTest < Pitchfork::IntegrationTest

test/integration/test_heartbeat_timeout.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'integration_test_helper'
 
 class HearbeatTimeoutTest < Pitchfork::IntegrationTest

test/integration/test_http_basic.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'integration_test_helper'
 
 class HttpBasicTest < Pitchfork::IntegrationTest

test/integration/test_info.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'integration_test_helper'
 
 class InfoTest < Pitchfork::IntegrationTest

test/integration/test_parser_error.rb

@@ -1,4 +1,5 @@
 # -*- encoding: binary -*-
+# frozen_string_literal: true
 
 require 'integration_test_helper'
 

test/integration/test_reap_logging.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'integration_test_helper'
 
 class ReapLoggingTest < Pitchfork::IntegrationTest

test/integration/test_reforking.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'integration_test_helper'
 
 class ReforkingTest < Pitchfork::IntegrationTest

test/integration/write-on-close.ru

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class WriteOnClose
   def each(&block)
     @callback = block