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.

Sign up for GitHub

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

Jump to bottom

Remove default value for cookies_domain #6440

Merged
merged 1 commit into from
Mar 2, 2017
Merged

Remove default value for cookies_domain #6440

merged 1 commit into from
Mar 2, 2017

Commits on Mar 2, 2017

  1. Remove default value for cookies_domain. Fixes bolt#6431 

    The only thing this did differently than PHP's default domain is removed
the www. prefix allowing the cookie to assigned to foo.com and www.foo.com.

It was decided this isn't a big enough use case to warrant keeping this logic.
Note that fixing this caching bug would mean moving the logic to 3 different
places (or 2 with larger refactoring).
    @CarsonF
    CarsonF committed Mar 2, 2017
    Configuration menu
    Copy the full SHA
    89f5d13 View commit details
    Browse the repository at this point in the history