diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2016-12-21 17:17:13 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2016-12-21 17:17:13 +0100 |
| commit | 7ae60ff0cc577997b9b754966a0f52649077b744 (patch) | |
| tree | ad354985346413a2178811620fbeefe1bab85b88 /lib | |
| parent | f4bbc2ba820277fc535c66233f6a94aea91c5a3b (diff) | |
| parent | 7f8aecf5391706858eb5714f2a906fbd3e775d25 (diff) | |
Merge pull request #1385 from Alkarex/favicon-user-agent
Add curl user-agent to retrieve favicon
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/favicons.php | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/lib/favicons.php b/lib/favicons.php index 6709f6745..d8c97964e 100644 --- a/lib/favicons.php +++ b/lib/favicons.php @@ -9,7 +9,7 @@ $default_favicon = PUBLIC_PATH . '/themes/icons/default_favicon.ico'; function download_favicon($website, $dest) { global $favicons_dir, $default_favicon; - syslog(LOG_DEBUG, 'FreshRSS Favicon discovery GET ' . $website); + syslog(LOG_INFO, 'FreshRSS Favicon discovery GET ' . $website); $favicon_getter = new \Favicon\Favicon(); $favicon_getter->setCacheDir($favicons_dir); $favicon_url = $favicon_getter->get($website); @@ -18,11 +18,12 @@ function download_favicon($website, $dest) { return @copy($default_favicon, $dest); } - syslog(LOG_DEBUG, 'FreshRSS Favicon GET ' . $favicon_url); + syslog(LOG_INFO, 'FreshRSS Favicon GET ' . $favicon_url); $c = curl_init($favicon_url); curl_setopt($c, CURLOPT_HEADER, false); curl_setopt($c, CURLOPT_RETURNTRANSFER, true); curl_setopt($c, CURLOPT_BINARYTRANSFER, true); + curl_setopt($c, CURLOPT_USERAGENT, 'FreshRSS/' . FRESHRSS_VERSION . ' (' . PHP_OS . '; ' . FRESHRSS_WEBSITE . ')'); $img_raw = curl_exec($c); $status_code = curl_getinfo($c, CURLINFO_HTTP_CODE); curl_close($c); @@ -34,6 +35,8 @@ function download_favicon($website, $dest) { fclose($file); return true; } + } else { + syslog(LOG_WARNING, 'FreshRSS Favicon GET ' . $favicon_url . ' error ' . $status_code); } return false; |