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

Additional HTTP error handling in get_article_content

This commit is contained in:
Shadowfacts 2019-06-30 21:31:24 -04:00
parent b686938bfa
commit e004b65263
Signed by: shadowfacts
GPG Key ID: 94A5AB95422746E5
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/frenzy/update_feeds.ex
View File

@ -199,7 +199,8 @@ defmodule Frenzy.UpdateFeeds do
{:ok, %HTTPoison.Response{status_code: 404}} -> {:ok, %HTTPoison.Response{status_code: 404}} ->
{:err, "404 not found"} {:err, "404 not found"}
{:ok, %HTTPoison.Response{status_code: 301, headers: headers}} -> {:ok, %HTTPoison.Response{status_code: status_code, headers: headers}}
when status_code == 301 or status_code == 302 ->
{"Location", new_url} = {"Location", new_url} =
Enum.find(headers, fn {name, _value} -> Enum.find(headers, fn {name, _value} ->
name == "Location" name == "Location"
@ -208,6 +209,9 @@ defmodule Frenzy.UpdateFeeds do
Logger.debug("Got 301 redirect from #{url} to #{new_url}") Logger.debug("Got 301 redirect from #{url} to #{new_url}")
get_article_content(new_url) get_article_content(new_url)
{:ok, %HTTPoison.Response{status_code: 403}} ->
{:err, "403 Forbidden"}
{:error, %HTTPoison.Error{reason: reason}} -> {:error, %HTTPoison.Error{reason: reason}} ->
{:err, reason} {:err, reason}
end end