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

core:archive infinite loop updating maxmind db #8707

Closed
iamsilvio opened this issue Sep 4, 2015 · 0 comments
Closed

core:archive infinite loop updating maxmind db #8707

iamsilvio opened this issue Sep 4, 2015 · 0 comments
Assignees
@sgiehl
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
2.15.0

Comments

@iamsilvio
Copy link

The following error will be logged in a infinite loop to the log file if the uri to geolite.maxmind.com could not be resolved

http.php line 430 seems to be always true

WARNING [2015-09-04 01:49:43] /piwik/core/Http.php(431): Warning - fread() expects parameter 1 to be resource, boolean given - Piwik 2.14.3
WARNING [2015-09-04 01:49:43] /piwik/core/Http.php(430): Warning - feof() expects parameter 1 to be resource, boolean given - Piwik 2.14.3

To reproduce just add a entry like 0.0.0.0 geolite.maxmind.com to the hosts file.
@sgiehl sgiehl added the Bug For errors / faults / flaws / inconsistencies etc. label Sep 5, 2015
@sgiehl sgiehl added this to the 2.15.0 milestone Sep 5, 2015
@sgiehl sgiehl self-assigned this Sep 5, 2015
@sgiehl sgiehl mentioned this issue Sep 5, 2015
Merged
@sgiehl sgiehl closed this as completed in 8485920 Sep 7, 2015
diosmosis added a commit that referenced this issue Sep 7, 2015
@diosmosis
Merge pull request #8714 from piwik/http_fopen_fix
7c9382c 
[fixes #8707] Fix for fopen method in http class if url is not reachable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sgiehl sgiehl

Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
2.15.0
Development

No branches or pull requests
2 participants
@sgiehl @iamsilvio