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

httpOnly cookies #2676

Merged
merged 1 commit into from Nov 11, 2014

Conversation

Projects
None yet
3 participants
@DimitarChristoff
Member

DimitarChristoff commented Nov 11, 2014

fixes #2664

@kentaromiura

This comment has been minimized.

Show comment
Hide comment
@kentaromiura
Member

kentaromiura commented Nov 11, 2014

Lgtm

@kentaromiura kentaromiura added this to the 1.5.2 milestone Nov 11, 2014

@kentaromiura

This comment has been minimized.

Show comment
Hide comment
@kentaromiura

kentaromiura Nov 11, 2014

Member

@ibolmo since it's a pretty simple change for us and it will let devs to have a big security improvement against simple XSS attacks (*) I've set the milestone for this to 1.5.2

(*) http://blog.codinghorror.com/protecting-your-cookies-httponly/

Member

kentaromiura commented Nov 11, 2014

@ibolmo since it's a pretty simple change for us and it will let devs to have a big security improvement against simple XSS attacks (*) I've set the milestone for this to 1.5.2

(*) http://blog.codinghorror.com/protecting-your-cookies-httponly/

arian added a commit that referenced this pull request Nov 11, 2014

@arian arian merged commit 2628cd5 into mootools:master Nov 11, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment