Merge pull request #1251 from sul-dlss/1250-timeout

Set an absurd HTTP timeout for IIIF requests; fixes #1250
sul-dlss · Aug 20, 2024 · 1ec34af · 1ec34af
2 parents e68389f + 39c5c9d
commit 1ec34af
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/models/iiif_image.rb b/app/models/iiif_image.rb
@@ -19,7 +19,7 @@ def initialize(stacks_file:, transformation:, base_uri: Settings.imageserver.bas
   def response
     with_retries max_tries: 3, rescue: [HTTP::ConnectionError] do
       benchmark "Fetch #{image_url}" do
-        HTTP.timeout(connect: 15)
+        HTTP.timeout(connect: 15, read_timeout: 5.minutes)
             .headers(user_agent: "#{HTTP::Request::USER_AGENT} (#{Settings.user_agent})")
             .use({ normalize_uri: { normalizer: lambda(&:itself) } })
             .get(image_url)

diff --git a/app/services/iiif_metadata_service.rb b/app/services/iiif_metadata_service.rb
@@ -47,7 +47,7 @@ def retrieve
       handle_response(
         # Disable url normalization as an upstream bug in addressable causes issues for `+`
         # https://github.com/sporkmonger/addressable/issues/386
-        HTTP.timeout(connect: 15)
+        HTTP.timeout(connect: 15, read_timeout: 5.minutes)
             .headers(user_agent: "#{HTTP::Request::USER_AGENT} (#{Settings.user_agent})")
             .use({ normalize_uri: { normalizer: lambda(&:itself) } }).get(@url)
       )