We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Plus signs in cookie values are converted to spaces.
My reading of rfc6265 is that plus signs:
cookie-octet = %x21 / %x23-2B / %x2D-3A / %x3C-5B / %x5D-7E
as the plus sign is a %2B
The function parseCookieHeader does a urldecode on the values here, and so the plus sign is converted to a space.
$data = "john=12345; plus_sign=plus+sign"; $result = parseCookieHeader($data); var_dump($data, $result);
As the plus sign is apparently a valid cookie character, it should be passed through unchanged.
Not sure how you could change this without a large BC break...
btw, probably relevant https://bugs.php.net/bug.php?id=78929
The text was updated successfully, but these errors were encountered:
Hack for plus signs being converted to plusses laminas/laminas-diacto…
9c84672
…ros#165.
@Danack thank you
Sorry, something went wrong.
Successfully merging a pull request may close this issue.
Bug Report
Plus signs in cookie values are converted to spaces.
My reading of rfc6265 is that plus signs:
as the plus sign is a %2B
Summary
Plus signs in cookie values are converted to spaces.
Current behavior
The function parseCookieHeader does a urldecode on the values here, and so the plus sign is converted to a space.
How to reproduce
$data = "john=12345; plus_sign=plus+sign";
$result = parseCookieHeader($data);
var_dump($data, $result);
Expected behavior
As the plus sign is apparently a valid cookie character, it should be passed through unchanged.
Not sure how you could change this without a large BC break...
btw, probably relevant
https://bugs.php.net/bug.php?id=78929
The text was updated successfully, but these errors were encountered: