sentientwaffle/node-favicon

Prefer <link rel=icon> to /favicon.ico

IvanSanchez opened this issue · 2 comments

IvanSanchez commented

As of now, node-favicon prefers to use /favicon.ico instead of parsing the webpage for a .

The W3C recommendation on the issue (http://www.w3.org/2005/10/howto-favicon) is that the is preferred.

jchook commented

Adding to this, you encounter some interesting edge cases in the wild:

  • Sites list an invalid or 404 <link rel="icon">, yet have a working /favicon.ico file (e.g. scholastic)
  • Sites list many <link rel="icon"> entries at different sizes (e.g. nbc)
wle8300 commented

See: https://www.npmjs.com/package/@meltwater/fetch-favicon

For a more robust solution than node-favicon