summaryrefslogtreecommitdiffstats
path: root/app/services/fetch_atom_service.rb
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2016-09-26 17:04:05 +0200
committerEugen Rochko <eugen@zeonfederated.com>2016-09-26 17:04:05 +0200
commit210362e66578dae32cbf169884d3c2905fe32cb7 (patch)
tree1065654c18ec90e8ec161c153f28799be8a9e463 /app/services/fetch_atom_service.rb
parentc6b0311b8626b42bc7e79e0195047a50e5b64dd1 (diff)
Improve redirect handling in FetchAtomService
Diffstat (limited to 'app/services/fetch_atom_service.rb')
-rw-r--r--app/services/fetch_atom_service.rb8
1 files changed, 4 insertions, 4 deletions
diff --git a/app/services/fetch_atom_service.rb b/app/services/fetch_atom_service.rb
index 57f789ada90..ada36b9f8c2 100644
--- a/app/services/fetch_atom_service.rb
+++ b/app/services/fetch_atom_service.rb
@@ -8,7 +8,7 @@ class FetchAtomService < BaseService
if response.mime_type == 'application/atom+xml'
return [url, fetch(url)]
elsif !response['Link'].blank?
- return process_headers(response)
+ return process_headers(url, response)
else
return process_html(fetch(url))
end
@@ -26,13 +26,13 @@ class FetchAtomService < BaseService
return [alternate_link['href'], fetch(alternate_link['href'])]
end
- def process_headers(response)
+ def process_headers(url, response)
Rails.logger.debug "Processing link header"
link_header = LinkHeader.parse(response['Link'])
alternate_link = link_header.find_link(['rel', 'alternate'], ['type', 'application/atom+xml'])
- return nil if alternate_link.nil?
+ return process_html(fetch(url)) if alternate_link.nil?
return [alternate_link.href, fetch(alternate_link.href)]
end
@@ -41,6 +41,6 @@ class FetchAtomService < BaseService
end
def http_client
- HTTP.timeout(:per_operation, write: 20, connect: 20, read: 50)
+ HTTP.timeout(:per_operation, write: 20, connect: 20, read: 50).follow
end
end