Fix icon uses all strategies instead of only the first success strategy

hyperjumptech · May 24, 2023 · 13aa9c5 · 13aa9c5
1 parent ff8ed8a
commit 13aa9c5
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.0.2]
+
+Fixed
+
+- Fix icon uses all strategies instead of only the first success strategy
+
 ## [1.0.1]
 
 Added

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@hyperjumptech/favicon-fetcher",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "A utility to fetch a website's favicon",
   "main": "dist/index.js",
   "scripts": {

diff --git a/src/get-favicon.ts b/src/get-favicon.ts
@@ -77,7 +77,10 @@ export async function getFavicon(
         return Buffer.from(ab)
       })
 
-      icon = result
+      if (result) {
+        icon = result
+        break
+      }
 
       // Else, use another strategy
       continue