fix(Downloaders): tolerate DNS lookup timeout and use caches

evshiron · Aug 21, 2017 · 434aea4 · 434aea4
1 parent bf3e44b
commit 434aea4
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/lib/Downloader.ts b/src/lib/Downloader.ts
@@ -67,12 +67,21 @@ export class Downloader extends DownloaderBase {
             }
         }
         catch(err) {
+
+            debug('in fetch', 'err', err);
+
             if(err.code === 'ENOTFOUND' && this.options.useCaches) {
+                console.info('DNS lookup fails, use local caches at this time.');
+                return path;
+            }
+            else if(err.code === 'EAI_AGAIN' && this.options.useCaches) {
+                console.info('DNS lookup timeout, use local caches at this time.');
                 return path;
             }
             else {
                 throw err;
             }
+
         }
 
         await this.download(url, filename, path, showProgress);

diff --git a/src/lib/FFmpegDownloader.ts b/src/lib/FFmpegDownloader.ts
@@ -76,12 +76,21 @@ export class FFmpegDownloader extends DownloaderBase {
             }
         }
         catch(err) {
+
+            debug('in fetch', 'err', err);
+
             if(err.code === 'ENOTFOUND' && this.options.useCaches) {
+                console.info('DNS lookup fails, use local caches at this time.');
+                return path;
+            }
+            else if(err.code === 'EAI_AGAIN' && this.options.useCaches) {
+                console.info('DNS lookup timeout, use local caches at this time.');
                 return path;
             }
             else {
                 throw err;
             }
+
         }
 
         await this.download(url, filename, path, showProgress);