search

ytti / oxidized-web

Web UI + RESTful API for Oxidized
118 stars 72 forks source link

support puma6 #239

Closed nickhilliard closed 1 year ago

nickhilliard commented 1 year ago

oxidized-web will run happily with any version of puma up to 5.6.5. However Rack::Handler.get(:puma) no longer works as a construct in puma 6.x.

This can be fixed by removing references to Rack::Handler::WEBrick, and calling the puma handler directly.

This references the problem in #238

mortzu commented 1 year ago

will this run on older version too?

nickhilliard commented 1 year ago

the patch works fine with older versions of puma. Tested on puma-3.11.4.