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

add system checks for auto_prepend_file and auto_append_file and session.auto_start is 0 #4152

Closed
robocoder opened this issue Sep 12, 2013 · 4 comments
Labels
duplicate For issues that already existed in our issue tracker and were reported previously. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Milestone

Comments

@robocoder
Copy link
Contributor

Should make sure session.auto_start is 0. We don't want sessions to be created for piwik.php requests.

auto_prepend_file and auto_append_file should be null. We don't want any additional overhead on piwik.php requests.

@mattab
Copy link
Member

mattab commented Feb 1, 2014

is auto_start really breaking Piwik? if so, sounds important to test against it indeed!

@mattab
Copy link
Member

mattab commented Feb 1, 2014

or maybe instead we explicitely disable it all the time in piwik.php?

@robocoder
Copy link
Contributor Author

Creating sessions takes time and resources. This impacts throughput in piwik.php requests and can lead to DoS (eg where session files are cleaned up by cron).

Session auto start and auto prepend happen before piwik.php is executed.

@robocoder robocoder added this to the 2.x - The Great Piwik 2.x Backlog milestone Jul 8, 2014
@mattab mattab changed the title Installation - more system checks add system checks for auto_prepend_file and auto_append_file and session.auto_start is 0 Oct 11, 2014
@mattab mattab modified the milestones: Long term, Mid term Oct 11, 2014
@mattab mattab modified the milestones: Long term, Mid term Dec 23, 2015
@mattab
Copy link
Member

mattab commented Jul 8, 2016

Diagnostic for session.auto_start exists.

@mattab mattab closed this as completed Jul 8, 2016
@mattab mattab added the duplicate For issues that already existed in our issue tracker and were reported previously. label Jul 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate For issues that already existed in our issue tracker and were reported previously. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Development

No branches or pull requests

2 participants