(fix) elixir-inspector#31 Don't crash on download failures

(fix) Require Logger
SiftLogic · Jun 29, 2023 · ab0bcbb · ab0bcbb
1 parent fef061f
commit ab0bcbb
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/lib/ua_inspector/downloader.ex b/lib/ua_inspector/downloader.ex
@@ -19,6 +19,8 @@ defmodule UAInspector.Downloader do
   alias UAInspector.Downloader.ShortCodeMapConverter
   alias UAInspector.ShortCodeMap
 
+  require Logger
+
   @client_hints [
     ClientHints.Apps,
     ClientHints.Browsers
@@ -106,8 +108,14 @@ defmodule UAInspector.Downloader do
   end
 
   defp download_file(remote, local) do
-    {:ok, content} = Config.downloader_adapter().read_remote(remote)
-
-    File.write!(local, content)
+    case Config.downloader_adapter().read_remote(remote) do
+      {:ok, content} ->
+        File.write!(local, content)
+        :ok
+
+      {:error, reason} ->
+        Logger.error("Failed to download file: #{reason}")
+        :ok
+    end
   end
 end