Patrik Ragnarsson

Results 301 comments of Patrik Ragnarsson

Warning sounds good to me, don't think we need/should make it fail (maybe you are just testing out the other mode temporary). If opt-in with `PUMA_UNUSED_CONFIG=warn` I don't think it...

> https://github.com/ioquatix/puma-switching-protocols If I change the status code from `101` to `200` the callback is invoked (I don't know why) Saw it here: https://github.com/puma/puma/blob/f59887602c66ad1f257ad57ab3aa27214b133c11/test/test_puma_server.rb#L580-L595

Will 6.0 address https://github.com/puma/puma/issues/2921?

> 2\. Should `WEB_CONCURRENCY` change I don't think it should change name, it could be breaking in the opposite way that [Puma 5.0 was](https://github.com/puma/puma/blob/15b079439222e489915380021b9dca7b9fa5ff65/5.0-Upgrade.md). Puma 5.0 introduced the use of...

> Things that will break: > > Rails: [actioncable/test/client_test.rb](https://github.com/rails/rails/blob/5fb3e721e70a13a7c0facf2d9175c15fb5132cc4/actioncable/test/client_test.rb#L76-L80) `Puma::Server#min_threads`, `Puma::Server#max_threads` used here too: https://github.com/teamcapybara/capybara/blob/f7ab0b5cd5da86185816c2d5c30d58145fe654ed/lib/capybara/registrations/servers.rb#L43

Sounds like a good idea to me

I think you need to share more about your local env @nateberkopec? As we aren't seeing this in CI?

Shouldn't be the case since v2.0.0 (https://github.com/cloudamqp/amqp-client.js/pull/22) Are you still using v1?

Due to this, I've been forced to switch to @eregon's fork of chruby that have #431: https://github.com/eregon/chruby/tree/do-no-set-gem-home It was needed because I have a computer with Apple silicon (M2), and...