Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Prevent websitePath from overwriting #6811

Closed
wants to merge 1 commit into from
Closed

Prevent websitePath from overwriting #6811

wants to merge 1 commit into from

Conversation

tristanlins
Copy link
Contributor

The $GLOBALS['TL_CONFIG']['websitePath'] may be set in localconfig.php, in this case some Contao functions, like System::setCookie() which still use $GLOBALS['TL_CONFIG']['websitePath'], so no cookie will work in case of wrong cookie path.

The $GLOBALS['TL_CONFIG']['websitePath'] may be set in localconfig.php, in this case some Contao functions, like System::setCookie()* which still use $GLOBALS['TL_CONFIG']['websitePath'] use a wrong path, so no cookie will work in case of wrong cookie path.

* https://github.com/contao/core/blob/master/system/modules/core/library/Contao/System.php#L462
@leofeyer leofeyer added this to the 3.2.9 milestone Mar 19, 2014
@leofeyer
Copy link
Member

Damn right you are :(

@leofeyer
Copy link
Member

Merged in 6b0d458.

@leofeyer leofeyer closed this Mar 21, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants