jshttp / cookie

HTTP server cookie parsing and serialization
MIT License
1.36k stars 159 forks source link

Issue with Cookie Value in Latest Update (v1.0.0) – Request for Fix #192

Closed bderen closed 1 month ago

bderen commented 1 month ago

We have encountered an issue in our service after the latest update. The problem arises when we set a cookie with the following value:

35RJyi41Wg3XyCXldWpoMQ,m4YmYn0CeWlHeL-JJM6DBZFoWHLuuuRoUDKKHirn7dg

This is causing unexpected behavior within the service, and we suspect the update may have affected how the cookie value is being processed.

Could you kindly investigate the issue and provide a fix at your earliest convenience?

blakeembrey commented 1 month ago

Do you have a reproduction or an error message? What is the issue? As far as I can tell this is working:

serialize('test', '35RJyi41Wg3XyCXldWpoMQ,m4YmYn0CeWlHeL-JJM6DBZFoWHLuuuRoUDKKHirn7dg');
'test=35RJyi41Wg3XyCXldWpoMQ%2Cm4YmYn0CeWlHeL-JJM6DBZFoWHLuuuRoUDKKHirn7dg'