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

Do not include .htaccess or robots.txt in build #887

Closed
robocoder opened this issue Jul 28, 2009 · 2 comments
Closed

Do not include .htaccess or robots.txt in build #887

robocoder opened this issue Jul 28, 2009 · 2 comments
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc. wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it.
Milestone

Comments

@robocoder
Copy link
Contributor

The build script should exclude .htaccess and robots.txt from the build, to not overwrite the user's local configuration.

e.g.,

@mattab
Copy link
Member

mattab commented Jul 28, 2009

I don't think there is any .htaccess in the build, appart from the one in config/ that is there for security purposes and should not suffer / overwrite a configuration.

The robots.txt is necessary otherwise the error.log/access.log gets full of 404 errors when bots try to load the robots.txt which has been an issue reported by several users in the past.

@robocoder
Copy link
Contributor Author

Quite right, I forgot ./.htaccess wasn't under version control.

re: robots,txt. I agree it's convenient, but the flip side is that (a) unzipping into an existing installation will overwrite local mods and (b) it isn't used when piwik is installed in a subfolder, eg http://example.com/piwik

@robocoder robocoder added this to the Piwik 0.4.3 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. wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it.
Projects
None yet
Development

No branches or pull requests

2 participants