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

Touch up the cookie logic a bit #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Touch up the cookie logic a bit #18

wants to merge 3 commits into from

Commits on May 11, 2015

  1. Don't kill cookies for URIs like "/page/a.small.town.in.nc/" 

    Untested RegEx.  YMMV.
    @BMDan
    BMDan committed May 11, 2015
    Configuration menu
    Copy the full SHA
    ca513d9 View commit details
    Browse the repository at this point in the history

Commits on May 12, 2015

  1. Only run cookie-related code is there's a cookie present 

    Based on varnishlog output, the "remove a ';' prefix" line was running on every request, *creating* a cookie for no good reason on cookie-less requests.  We now only run cookie manipulation is a cookie was originally present.
    @BMDan
    BMDan committed May 12, 2015
    Configuration menu
    Copy the full SHA
    abd5787 View commit details
    Browse the repository at this point in the history

  2. Merge pull request #1 from BMDan/cookie-cache-rework 

    Only run cookie-related code is there's a cookie present
    @BMDan
    BMDan committed May 12, 2015
    Configuration menu
    Copy the full SHA
    44dd64e View commit details
    Browse the repository at this point in the history