Explicitly require rack/common_logger if :Verbose is true

[wildmaples]

Description

Hello!

I found an issue where I had to require 'rack' or more specifically, 'rack/common_logger' when verbose is set to true, :Verbose => true.

Take this example:

require 'rack/handler/puma'

app = -> environment {[200, {}, []]}
Rack::Handler::Puma.run(app, :Verbose => true)

This is the exception:

Traceback (most recent call last):
       10: from /usr/bin/irb:23:in `<main>'
        9: from /usr/bin/irb:23:in `load'
        8: from /Library/Ruby/Gems/2.6.0/gems/irb-1.0.0/exe/irb:11:in `<top (required)>'
        7: from (irb):3
        6: from /Users/mapleong/.gem/ruby/2.6.0/gems/puma-5.2.0/lib/rack/handler/puma.rb:63:in `run'
        5: from /Users/mapleong/.gem/ruby/2.6.0/gems/puma-5.2.0/lib/rack/handler/puma.rb:32:in `config'
        4: from /Users/mapleong/.gem/ruby/2.6.0/gems/puma-5.2.0/lib/rack/handler/puma.rb:32:in `new'
        3: from /Users/mapleong/.gem/ruby/2.6.0/gems/puma-5.2.0/lib/puma/configuration.rb:154:in `initialize'
        2: from /Users/mapleong/.gem/ruby/2.6.0/gems/puma-5.2.0/lib/puma/configuration.rb:161:in `configure'
        1: from /Users/mapleong/.gem/ruby/2.6.0/gems/puma-5.2.0/lib/rack/handler/puma.rb:34:in `block in config'
NameError (uninitialized constant Rack::CommonLogger)

Users shouldn't have to require rack/common_logger on their end so I added a require "rack/common_logger" in the block where Verbose is true.

Questions:

• Do you agree with this change? If so,
  • What is the best way to test this? I didn't find a good place to test it in test_rack_handler.rb
  • Should I add this change to the changelog / History.md?

Your checklist for this pull request

• I have reviewed the guidelines for contributing to this repository.
• I have added an entry to History.md if this PR fixes a bug or adds a feature. If it doesn't need an entry to HISTORY.md, I have added [changelog skip] or [ci skip] to the pull request title.
• I have added appropriate tests if this PR fixes a bug or adds a feature. (pending question)
• My pull request is 100 lines added/removed or less so that it can be easily reviewed.
• If this PR doesn't need tests (docs change), I added [ci skip] to the title of the PR.
• If this closes any issues, I have added "Closes #issue" to the PR description or my commit messages.
• I have updated the documentation accordingly.
• All new and existing tests passed, including Rubocop. (pending)

[nateberkopec]

Makes sense to me, thanks for the catch 👍