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 req.session.cookie.hasLongExpires because YAGNI #870

Merged
merged 2 commits into from Dec 11, 2013
Merged

remove req.session.cookie.hasLongExpires because YAGNI #870

merged 2 commits into from Dec 11, 2013

Commits on Dec 11, 2013

  1. add a test for req.seesion.cookie.maxAge changed to large value

    @undoZen
    undoZen committed Dec 11, 2013
    Copy the full SHA
    f0ba08b View commit details
    Browse the repository at this point in the history

  2. remove req.session.cookie.hasLongExpires because YAGNI 

    It's not only not needed, but also cause problems.
senchalabs#859
    @undoZen
    undoZen committed Dec 11, 2013
    Copy the full SHA
    06ef585 View commit details
    Browse the repository at this point in the history