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.

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

Change cookie name to SULUSESSID #21

Merged

Conversation

@alexander-schranz
Copy link
Member

alexander-schranz commented Nov 14, 2019

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets fixes -
Related issues/PRs -
License MIT
Documentation PR -

What's in this PR?

Change cookie name to SULUSESSID to avoid conflict with PHPSESSID.

Why?

Avoids conflicts to applications running on the same domain overwriting this cookie session.

config/packages/framework.yaml Outdated Show resolved Hide resolved
@danrot
danrot approved these changes Nov 15, 2019
@danrot danrot merged commit e7fdd8d into sulu:release/2.0 Nov 15, 2019
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@alexander-schranz alexander-schranz deleted the alexander-schranz:enhancement/change-cookie-name branch Nov 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.