puma / puma

A Ruby/Rack web server built for parallelism

Home Page:https://puma.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Using rack 3. Receive deprecation warning.

mmarvb8h opened this issue · comments

Describe the bug
deprecation warning: "Rack::Handler is deprecated and replaced by Rackup::Handler"

Spent a little time to determine the issue. The issue seems to be when rackup.rb requires puma.rb. puma.rb does a require on rack/handler.rb on about line 3. rack/handler.rb throws deprecation warning.

Expected behavior
App should start without deprecation warning.

Desktop (please complete the following information):

  • OS: Mac M1 arm
  • Puma Version 6.0.2
  • rack version 3.0.3

~Marvin Foster
email: mmarvb7@gmail.com or marvinfoster@finsync.com

Can you post complete code and steps to reproduce?

I'm guessing you just need rack 3+ in the Gemfile or otherwise loaded for this to occur.

I think the issue is when using both rack 3 and rackup (rackup was included with rack 2)

See jeremyevans/roda#301 (comment)

When an app is started with the rackup bin file, we're using Rack::Handler, but with rack 3, the namespace is Rackup::Handler. I.think...

I've got a patch which works when using the rackup bin, but I need to fix the tests...

@mmarvb8h Don't know if you can easily test a PR, but I believe PR #3061 fixes this issue.

Sorry been focused on elsewhere. since all does seem to work. i'm comfortable with warning for now. since i generally use puma anyway i can start puma directly. in order for me to push to a "prod-like" state i needed to verify cause. when pushed fix is verified and released i'll just update puma. thx...