Include Max-Age in Set-Cookie header #429

Merged
merged 1 commit into from Nov 2, 2012

Conversation

Projects
None yet
2 participants
Contributor

matthewtoast commented Sep 8, 2012

Allows :max_age to be passed to set_cookie_header!. If passed, includes max-age in the Set-Cookie header. Sets max-age=0; when the cookie header is deleted.

Apologies, since I know you just closed a similar pull request. At least two folks seem to think it worthwhile. :) Let me know could be improved or tested.

(Re: browsers, one source says IE will ignore max-age and fall back to expires.)

raggi added a commit that referenced this pull request Nov 2, 2012

Merge pull request #429 from matthewtoast/master
Allow Max-Age to be set in Set-Cookie header

@raggi raggi merged commit 74e0acd into rack:master Nov 2, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment