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

Enabled httpOnly for cookies (#2716) #3895

Merged
merged 1 commit into from Mar 29, 2018

Conversation

Projects
None yet
2 participants
@gfr
Contributor

gfr commented Mar 29, 2018

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets #2716
Refs tickets #3454
License MIT
Changelog updated yes

Description

Removed legacy code to enable httpOnly setting for cookies

Todos

  • Tests
  • Documentation
  • Changelog

@Guite Guite added this to the 1.5.6 milestone Mar 29, 2018

@Guite

This comment has been minimized.

Member

Guite commented Mar 29, 2018

Thank you Gabriel!

@Guite Guite merged commit eec0bce into zikula:1.5 Mar 29, 2018

2 of 4 checks passed

SensioLabsInsight Code quality below expectations.
Details
continuous-integration/styleci/pr Issues have been identified with 2 files
Details
clahub All contributors have signed the Contributor License Agreement.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Guite Guite referenced this pull request Mar 29, 2018

Closed

second cookie for ajax? #3454

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment