fixed URL resolving for same URL in browser

bpampuch · Jan 3, 2025 · 9974c93 · 9974c93
1 parent 507314d
commit 9974c93
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- Fixed URL resolving for same URL in browser
+
 ## 0.3.0-beta.15 - 2025-01-01
 
 - Reverted to the original `pdfkit` package, moving away from `@foliojs-fork`

diff --git a/src/browser-extensions/URLBrowserResolver.js b/src/browser-extensions/URLBrowserResolver.js
@@ -53,12 +53,17 @@ class URLBrowserResolver {
 		if (!this.resolving[url]) {
 			this.resolving[url] = new Promise((resolve, reject) => {
 				if (url.toLowerCase().indexOf('https://') === 0 || url.toLowerCase().indexOf('http://') === 0) {
-					fetchUrl(url, headers).then(buffer => {
-						this.fs.writeFileSync(url, buffer);
+					if (this.fs.existsSync(url)) {
+						// url was downloaded earlier
 						resolve();
-					}, result => {
-						reject(result);
-					});
+					} else {
+						fetchUrl(url, headers).then(buffer => {
+							this.fs.writeFileSync(url, buffer);
+							resolve();
+						}, result => {
+							reject(result);
+						});
+					}
 				} else {
 					// cannot be resolved
 					resolve();