Prevent AccountView and instance.get_or_update_favicon fails
authorHaelwenn (lanodan) Monnier <contact@hacktivis.me>
Fri, 4 Sep 2020 05:52:22 +0000 (07:52 +0200)
committerHaelwenn (lanodan) Monnier <contact@hacktivis.me>
Sat, 5 Sep 2020 03:43:50 +0000 (05:43 +0200)
lib/pleroma/instances/instance.ex

index 711c42158b1580b1a88da2c1f26cdadbc715fa36..ef5d17de47cad1cc52bff725dfc0b3f6e139039a 100644 (file)
@@ -145,6 +145,8 @@ defmodule Pleroma.Instances.Instance do
 
       favicon
     end
+  rescue
+    _ -> nil
   end
 
   defp scrape_favicon(%URI{} = instance_uri) do
@@ -159,7 +161,8 @@ defmodule Pleroma.Instances.Instance do
              |> Floki.attribute("link[rel=icon]", "href")
              |> List.first(),
            favicon <- URI.merge(instance_uri, favicon_rel) |> to_string(),
-           true <- is_binary(favicon) do
+           true <- is_binary(favicon),
+           true <- String.length(favicon) <= 255 do
         favicon
       else
         _ -> nil