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

Fix error session already started #14914

Merged
merged 2 commits into from Oct 2, 2019
Merged

Fix error session already started #14914

merged 2 commits into from Oct 2, 2019

Conversation

tsteur
Copy link
Member

@tsteur tsteur commented Sep 27, 2019

Got this error when going on eg index.php?module=PrivacyManager&action=privacySettings&idSite=1&period=week&date=2019-09-20 and a session was started by another tool. That page uses NONCE which then uses SessionNamespace which then wants to start the session even though it was already started.

Got this error when going on eg  `index.php?module=PrivacyManager&action=privacySettings&idSite=1&period=week&date=2019-09-20` and a session was started by another tool. That page uses NONCE which then uses SessionNamespace which then wants to start the session even though it was already started.
I don't think I can write a test for it but need to see.
@tsteur tsteur added the Pull Request WIP Indicates the current pull request is still work in progress and not ready yet for a review. label Sep 27, 2019
@tsteur tsteur added this to the 3.12.0 milestone Sep 27, 2019
@tsteur tsteur added Needs Review PRs that need a code review and removed Pull Request WIP Indicates the current pull request is still work in progress and not ready yet for a review. labels Sep 29, 2019
@tsteur tsteur merged commit db30cea into 3.x-dev Oct 2, 2019
@tsteur tsteur deleted the sessionalreadystarted branch October 2, 2019 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant