-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Javascript inclusion when bot visits. #84
Comments
At first view it looks like removing https://github.com/pluginkollektiv/statify/blob/master/inc/statify_frontend.class.php#L268-L270 would fix the problem, right? |
Yes, I think so @patrickrobrecht |
Unfortunately removing https://github.com/pluginkollektiv/statify/blob/master/inc/statify_frontend.class.php#L268-L270 is not the correct solution. After removing this, JS tracking will log 404 errors for example (because |
Changed the rules for appending JS snippet to avoid caching problems.
Changed the rules for appending JS snippet to avoid caching problems.
Changed the rules for appending JS snippet to avoid caching problems.
Javascript is not included in a cached page, when a bot was the first visitor.
As far as I can see, the problem is the interplay between those two areas:
https://github.com/pluginkollektiv/statify/blob/master/inc/statify_frontend.class.php#L268-L270
https://github.com/pluginkollektiv/statify/blob/master/inc/statify_frontend.class.php#L141-L145
The text was updated successfully, but these errors were encountered: