-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove servers #1658
Remove servers #1658
Conversation
@tenderlove I could not remove the CGI handler without breaking a ton of specs, so I have left it in place for now. However, one thing it did bring attention to is how ancient and "specific" the I have a moderately strong feeling to remove rackup to it's own gem, along with the remaining handlers and tests. Good because:
Bad because:
Thoughts? |
I'm against moving rackup to another gem. I think you would be surprised how many users use rackup. Have you setup a gem for the removed servers? If not, I expect we'll receive pushback when Rack 3.0 is released. |
We discussed this and you said not to bother.
|
Ah, OK. :) |
I guess we should let @macournoyer know this has finally landed. I was added as a maintainer of thin but I have little time to support it. That being said, I think there are a few options to consider:
While we would try to minimise breaking changes to server code in Rack 3 (nightmare for server maintainers to support Rack 1 - 3 if there are breaking changes), it's likely some changes will be required. So, if we can't find maintainer/support for thin, I recommend archive it to avoid users getting frustrated. |
No worries, I’ll take care of update Thin. Thanks for the heads up! |
No description provided.