rich media: add try/rescue to ensure we catch parsing and fetching failures

2019-01-27 12:21:05 +00:00 · 2019-01-27 12:21:05 +00:00 · de42646634
parent 5d895093fd
commit de42646634
1 changed files with 7 additions and 2 deletions
--- a/lib/pleroma/web/rich_media/parser.ex
+++ b/lib/pleroma/web/rich_media/parser.ex
@ -21,9 +21,14 @@ defmodule Pleroma.Web.RichMedia.Parser do
  end

  defp parse_url(url) do
-    {:ok, %Tesla.Env{body: html}} = Pleroma.HTTP.get(url)
+    try do
+      {:ok, %Tesla.Env{body: html}} = Pleroma.HTTP.get(url)

-    html |> maybe_parse() |> get_parsed_data()
+      html |> maybe_parse() |> get_parsed_data()
+    rescue
+      _e ->
+        {:error, "Parsing error"}
+    end
  end

  defp maybe_parse(html) do