-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Fix Puma::StateFile#load
incompatibility
#2810
Conversation
df4be13
to
350b59c
Compare
Until 5.5.2, empty values were read as `nil`. Since 5.6.0, empty values are read as `""`. This commit fixes the incompatibility introduced in 5.6.0.
350b59c
to
e5c7f05
Compare
Thanks. Pounding head against desk... I'm sorry for the bug, as I introduced it. Locally, I've always used scripts that worked on Windows & Ubuntu, and Windows doesn't really support signals. Hence, I had no |
@MSP-Greg Hello. I'm waiting for the release of this patch. Is there any schedule for it? |
My bad. We're gonna push 6.0 soon but I'll do a 5.6.5 bugfix release as well. |
@nateberkopec I'm sorry to bother you, but do you think it will take much longer to release 5.6.5? |
Sorry, been busy with kid and job. I hope to start doing some releases this week. |
@nateberkopec Thank you for your update. I look forward to the release. |
@nateberkopec Could you release 5.6.5? I want to update puma because of CVE-2022-24790, but this incompatibility is a blocker. |
5.6.5 has been released! |
Great, thank you @nateberkopec ! |
Description
Until 5.5.2, empty values were read as
nil
.Since 5.6.0, empty values are read as
""
.This PR fixes the incompatibility.
This incompatibility affects here for example.
Your checklist for this pull request
[ci skip]
to the title of the PR.#issue
" to the PR description or my commit messages.