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

HTTP Response: Cookies #3

Closed
roydejong opened this issue Jul 7, 2015 · 1 comment · Fixed by #13
Closed

HTTP Response: Cookies #3

roydejong opened this issue Jul 7, 2015 · 1 comment · Fixed by #13
Assignees
@roydejong
Labels
enhancement
Milestone
v0.1

Comments

@roydejong
Copy link
Owner

It should be possible to send cookies using the Http\Response class.

Proposed API:

  • Request::sendCookie($key, $value, $ttl, [..])
  • Request::deleteCookie($key)
@roydejong roydejong self-assigned this Jul 7, 2015
@roydejong roydejong added this to the v0.1 milestone Jul 7, 2015
roydejong added a commit that referenced this issue Jul 7, 2015
@roydejong
[#3] Request: Add support for reading cookies (getCookie, getCookies,…
3b37475 
… setCookieData)
@roydejong
Copy link
Owner Author

Woops, got issues mixed up.

@roydejong roydejong reopened this Jul 7, 2015
roydejong added a commit that referenced this issue Jul 12, 2015
@roydejong
[#3] Cookies: Implement base cookie class / config API
66f607a
roydejong added a commit that referenced this issue Jul 12, 2015
@roydejong
[#3, #10] Response: Add support for configuring + sending Cookies. Re…
2448e25 
…factor code + phpdocs.
roydejong added a commit that referenced this issue Jul 12, 2015
@roydejong
[#3] Unit tests: Add tests for Cookie class; fix Cookie::setExpireOnS…
cac42e3 
…ession()
roydejong added a commit that referenced this issue Jul 12, 2015
@roydejong
[#3] Unit tests: Add tests for Response cookie management / sending
cfb500d
roydejong added a commit that referenced this issue Jul 12, 2015
@roydejong
[#3] Cookies: Implement Response::unsetCookie($name) plus tests
2bf00a8
@roydejong roydejong mentioned this issue Jul 12, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@roydejong roydejong

Labels
enhancement
Projects
None yet
Milestone
v0.1
Development

Successfully merging a pull request may close this issue.

HTTP Response: Get and Set cookies roydejong/Enlighten
1 participant
@roydejong