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

SecurityInfo Plugin: Cannot get rid of warning #4006

Closed
timo-bes opened this issue Jun 12, 2013 · 2 comments
Closed

SecurityInfo Plugin: Cannot get rid of warning #4006

timo-bes opened this issue Jun 12, 2013 · 2 comments
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@timo-bes
Copy link
Member

In the SecurityInfo plugin, the Session > save_path test shows a warning (yellow). After debugging, I found out that the reason it shows the warning is that the common tempdir occurs in the session save_path. But the warning is wrong because the tempdir is /tmp/ and the session save_path is set by Piwik to /path/to/piwik/tmp/sessions/, which contains /tmp/. So there's no way to get rid of the warning by configuration. I think the test should be changed to only check whether the save_path starts with the common tempdir, not whether it contains it.

How do we deal with that? Do we open a ticket on their repo?

@mattab
Copy link
Member

mattab commented Jun 18, 2013

I think the project is not maintained anymore, so it would be OK to hot-fix our copy of the lib.

@halfdan
Copy link
Member

halfdan commented Sep 9, 2013

@timo-bes timo-bes added this to the 2.x - The Great Piwik 2.x Backlog milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

3 participants