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

Jump to bottom

web requests check for console file readability #342

Closed
craue opened this issue Jul 3, 2014 · 2 comments
Closed

web requests check for console file readability #342

craue opened this issue Jul 3, 2014 · 2 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. worksforme The issue cannot be reproduced and things work as intended.

Comments

@craue
Copy link
Contributor

craue commented Jul 3, 2014

After updating from 2.3.0 to 2.4.0 I'm getting this message in my Apache error log file:
<pre>ERROR Piwik\Tracker[2014-07-03 08:39:20] [1c930] The Piwik URL https://DOMAIN/piwik/index.php does not seem to be pointing to a Piwik server. Response was &#039;Could not open input file: /PATH/piwik/vendor/piwik/piwik/console &#039;.</pre>.

The console file exists and has deliberately 700 for root:root only, as I don't want to use that file and also don't want to expose it via the web server. Just deleting it yields the same message.

So what's the deal with a check for being able to read the console file while requests are made through the web server?

Apart from that, the time in the error message is wrong. It should be 10:39 (local time when the error occured) instead of 08:39.
@craue
Copy link
Contributor Author

craue commented Jul 11, 2014

I noticed that archiving is done by calling a console command, thus the file is needed. So I worked around the issue with a custom .htaccess file in the piwik folder to deny web access to the console file while keeping its permissions at 755. Should this be added to the repo?

@mattab
Copy link
Member

mattab commented Aug 3, 2014

I don't think it's necessary to add it in htaccess, as most people are fine with the console (since it only works in CLI).
Thanks for the suggestion.

@mattab mattab closed this as completed Aug 3, 2014
@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Sep 20, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc. worksforme The issue cannot be reproduced and things work as intended.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mattab @craue