shadowfacts
/
frenzy
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix feed update failing on missing favicon

This commit is contained in:
Shadowfacts 2020-08-14 21:56:38 -04:00
parent bd42073e24
commit 04ffe0036d
Signed by: shadowfacts
GPG Key ID: 94A5AB95422746E5
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/frenzy/task/fetch_favicon.ex
View File

@ -25,7 +25,8 @@ defmodule Frenzy.Task.FetchFavicon do
favicon_url =
fetch_favicon_url_from_webpage(site_url) || URI.merge(site_url, "/favicon.ico")
with %Feed{favicon_url: old_url} when old_url != favicon_url <- feed,
with true <- is_binary(favicon_url),
%Feed{favicon_url: old_url} when old_url != favicon_url <- feed,
{:ok, favicon_data} <- fetch_favicon_data(favicon_url) do
changeset =
Feed.changeset(feed, %{