Skip to content

cookie: compare cookie prefixes case insensitively#9863

Closed
bagder wants to merge 2 commits intomasterfrom
bagder/cookie-case-prefix
Closed

cookie: compare cookie prefixes case insensitively#9863
bagder wants to merge 2 commits intomasterfrom
bagder/cookie-case-prefix

Conversation

@bagder
Copy link
Member

@bagder bagder commented Nov 7, 2022

Adapted to language in rfc6265bis draft-11.

Adapted to language in rfc6265bis draft-11.
@bagder bagder added the HTTP label Nov 7, 2022
Copy link
Member

@danielgustafsson danielgustafsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would probably be good to change a test case to have a mixed-case prefix just to keep us honest.

Copy link
Member

@danielgustafsson danielgustafsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bagder bagder closed this in 14061f7 Nov 8, 2022
@bagder bagder deleted the bagder/cookie-case-prefix branch November 8, 2022 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants