Merge pull request #721 from microlinkhq/next

fix(logo-favicon): pass resolveFaviconUrl
microlinkhq · Jul 25, 2024 · 23f8080 · 23f8080
2 parents 0a06dd8 + 92d1030
commit 23f8080
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/packages/metascraper-logo-favicon/src/index.js b/packages/metascraper-logo-favicon/src/index.js
@@ -241,7 +241,11 @@ module.exports = ({
     logo: [
       toLogo(async ($, url) => {
         const sizes = getSizes($, sizeSelectors, url)
-        return await pickFn(sizes, { gotOpts, pickBiggerSize })
+        return await pickFn(sizes, {
+          resolveFaviconUrl,
+          gotOpts,
+          pickBiggerSize
+        })
       }),
       ({ url }) => getLogo(normalizeUrl(url)),
       rootFavicon