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
As per Chrome requirement:
A cookie associated with a cross-site resource at http://cloudflare.com/ was set without the SameSite attribute.
SameSite
In order to be able to use cookies for browsers language detection, we need the "sameSite" option while setting the "i18n_redirected" cookie.
Add an extra option "sameSite", boolean value.
detectBrowserLanguage: { ... sameSite: true ... },
The text was updated successfully, but these errors were encountered:
@Borisstoy I think it would make sense to create cookie with SameSite=Lax attribute by default. I don't really see why would we need an option for it.
SameSite=Lax
Sorry, something went wrong.
@rchl , indeed.
fix: set sameSite=Lax option for detected-language cookie
8d84986
Avoids warnings triggered by browsers related to cookie not having sameSite attribute. More details at https://web.dev/samesite-cookies-explained/ Resolves #516
this issue still persists
Successfully merging a pull request may close this issue.
What problem does this feature solve?
As per Chrome requirement:
A cookie associated with a cross-site resource at http://cloudflare.com/ was set without the
SameSite
attribute.In order to be able to use cookies for browsers language detection, we need the "sameSite" option while setting the "i18n_redirected" cookie.
What does the proposed changes look like?
Add an extra option "sameSite", boolean value.
detectBrowserLanguage: {
...
sameSite: true
...
},
The text was updated successfully, but these errors were encountered: