Skip to content
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

Warn on invalid cookie name #12799

Open
benmccann opened this issue Oct 12, 2024 · 0 comments · May be fixed by #12806
Open

Warn on invalid cookie name #12799

benmccann opened this issue Oct 12, 2024 · 0 comments · May be fixed by #12806
Assignees

Comments

@benmccann
Copy link
Member

Describe the problem

Latest version of cookie library accepts different cookie names. Need to tell anyone who is setting invalid name to stop so that we can upgrade cookie library in Kit 3.0

Describe the proposed solution

use regex from https://github.com/jshttp/cookie/pull/167/files

put it here:

set(name, value, options) {

Alternatives considered

No response

Importance

nice to have

Additional Information

No response

@eltigerchino eltigerchino linked a pull request Oct 14, 2024 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants