Skip to content
This repository has been archived by the owner on Feb 4, 2021. It is now read-only.

Correct language cookie path #303

Closed
johanjanssens opened this issue Mar 3, 2016 · 0 comments · Fixed by #305
Closed

Correct language cookie path #303

johanjanssens opened this issue Mar 3, 2016 · 0 comments · Fixed by #305
Labels
Milestone

Comments

@johanjanssens
Copy link
Member

The current implementation in plugins/system/languagefilter fails to correctly set the cookie path. This results in multiple language cookies for different paths.

For example, if you enable languages and add a few submenu items, you will see that a language cookie is being set on every sublevel. See this screenshot:

screen shot 2016-03-03 at 16 50 05

Use the basepath for each application (site|admin) so the language cookie is managed only once per app.

Example:

https://github.com/nooku/nooku-framework/blob/master/code/libraries/koowa/libraries/dispatcher/authenticator/csrf.php#L125

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant