most of the favicons are missing

[kinlong322]

for example, the generated iconURI is: "iconURI": "http://www.google.com/s2/favicons?domain=http://techcrunch.com/search/%q" this resulted in Bad request Error 400.

Instead, the query string of the iconURI should be trimmed off, leaving just the domain name for correctly fetching icon: http://www.google.com/s2/favicons?domain=http://techcrunch.com/

Thank you for this great add-on. I use it all the time.

[Drugoy]

Why not use the url to the site's icon? http://techcrunch.com/favicon.ico

[fix-fix]

@Drugoy because some websites don't use /favicon.ico scheme, e.g.: https://instagram.com/favicon.ico

[algkmn]

@fix-fix https://instagramstatic-a.akamaihd.net/bluebar/5ba3356/images/ico/favicon.ico

[fix-fix]

@algkmn I know they have some favicon somewhere. I was speaking about '/favicon.ico' scheme not being the universal approach, therefore this project should depend on external favicon service.

[iakoboss7]

any chance this will get fixed?