bitwalker / distillery

Simplify deployments in Elixir with OTP releases!
MIT License
2.97k stars 398 forks source link

Prevents generating cookies that contain commas #663

Closed heymackey closed 5 years ago

heymackey commented 5 years ago

Summary of changes

Prevents Distillery from generating cookies that contain commas. In our experience with a cookie that contained a comma, our nodes had issues connecting. When we replaced the comma with another character, the nodes were able to connect properly.

Checklist

Licensing/Copyright

By submitting this PR, you agree to the following statement, please read before submission!

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for Distillery (the "Contribution"). My Contribution is licensed under the MIT License.

bitwalker commented 5 years ago

Wow, can't believe we've never hit that before! Thanks for the PR!