Merge pull request #3396 from mikiher/custom-provider-try-catch

Add a try-catch block around custom provider search
advplyr · Sep 9, 2024 · 7a0cd1e · 7a0cd1e
2 parents ac3277d + 80685af
commit 7a0cd1e
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/server/finders/BookFinder.js b/server/finders/BookFinder.js
@@ -202,10 +202,14 @@ class BookFinder {
    * @returns {Promise<Object[]>}
    */
   async getCustomProviderResults(title, author, isbn, providerSlug) {
-    const books = await this.customProviderAdapter.search(title, author, isbn, providerSlug, 'book', this.#providerResponseTimeout)
-    if (this.verbose) Logger.debug(`Custom provider '${providerSlug}' Search Results: ${books.length || 0}`)
-
-    return books
+    try {
+      const books = await this.customProviderAdapter.search(title, author, isbn, providerSlug, 'book', this.#providerResponseTimeout)
+      if (this.verbose) Logger.debug(`Custom provider '${providerSlug}' Search Results: ${books.length || 0}`)
+      return books
+    } catch (error) {
+      Logger.error(`Error searching Custom provider '${providerSlug}':`, error)
+      return []
+    }
   }
 
   static TitleCandidates = class {