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.

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

Complete removal of cookie if it was set to null #293

Merged
merged 1 commit into from Aug 21, 2013

Conversation

Projects
None yet
2 participants
@sshamov
Contributor

sshamov commented Aug 21, 2013

No description provided.

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Aug 21, 2013

Member

Thanks, looks useful. I'll also add a brief doc comment mentioning this possibility.

Member

s-ludwig commented Aug 21, 2013

Thanks, looks useful. I'll also add a brief doc comment mentioning this possibility.

s-ludwig added a commit that referenced this pull request Aug 21, 2013

Merge pull request #293 from sshamov/master
Complete removal of cookie if it was set to null

@s-ludwig s-ludwig merged commit 7cdd77d into vibe-d:master Aug 21, 2013

s-ludwig added a commit that referenced this pull request Aug 21, 2013

Add parameter documentation for HTTPServerResponse.setCookie and use …
…"maxAge" instead of "expires" for clearing a cookie.

"expires" is not a standard field and the date/time format used was not matching the one commonly used for that field. "MaxAge=0" is the standard way to clear a cookie, so this is used here. See also #293.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment