chore: Log OpenGraph fetching errors (#6066)

wireapp · Mar 12, 2019 · 12588ee · 12588ee
1 parent 1ee04df
commit 12588ee
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/script/links/LinkPreviewRepository.js b/src/script/links/LinkPreviewRepository.js
@@ -148,19 +148,26 @@ class LinkPreviewRepository {
       return window
         .openGraphAsync(link)
         .then(mergeOpenGraphData)
-        .catch(() => Promise.resolve());
+        .catch(error => {
+          this.logger.warn(`Error while fetching OpenGraph data: ${error.message}`);
+          return Promise.resolve();
+        });
     }
 
     return new Promise(resolve => {
       return window
         .openGraph(link, (error, data) => {
           if (error) {
+            this.logger.warn(`Error while fetching OpenGraph data: ${error.message}`);
             resolve();
           }
 
           resolve(mergeOpenGraphData(data));
         })
-        .catch(resolve);
+        .catch(error => {
+          this.logger.warn(`Error while fetching OpenGraph data: ${error.message}`);
+          resolve(error);
+        });
     });
   }