Allow downloading subject/classification dumps for 24 hours

app/models/medium.rb

@@ -55,19 +55,19 @@ def url_for_format(format)
     end
   end
 
-  def put_url
+  def put_url(opts = {})
     if external_link
       src
     else
-      MediaStorage.put_path(src, indifferent_attributes)
+      MediaStorage.put_path(src, indifferent_attributes.merge(opts))
     end
   end
 
-  def get_url
+  def get_url(opts = {})
     if external_link
       src
     else
-      MediaStorage.get_path(src, indifferent_attributes)
+      MediaStorage.get_path(src, indifferent_attributes.merge(opts))
     end
   end
 

app/views/classification_data_mailer/classification_data.html.erb

@@ -1,3 +1,3 @@
 <p>Classification data for <%= @project.display_name %> is ready</p>
 
-<p><a href="<%= @url %>">Click to download your data</a>. This link will be valid for 1 hour.</p>
+<p><a href="<%= @url %>">Click to download your data</a>. This link will be valid for 24 hours.</p>

app/views/classification_data_mailer/classification_data.text.erb

@@ -1,3 +1,3 @@
 Classification data for <%= @project.display_name %> is ready
 
-Download at <%= @url %>. This link will be valid for 1 hour.
+Download at <%= @url %>. This link will be valid for 24 hours.

app/views/subject_data_mailer/subject_data.html.erb

@@ -1,4 +1,3 @@
 <p>Subject data for <%= @project.display_name %> is ready</p>
 
-<p><a href="<%= @url %>">Click to download your data</a>. This link will be valid for 1 hour.</p>
-
+<p><a href="<%= @url %>">Click to download your data</a>. This link will be valid for 24 hours.</p>

app/views/subject_data_mailer/subject_data.text.erb

@@ -1,3 +1,3 @@
 Subject data for <%= @project.display_name %> is ready
 
-Download at <%= @url %>. This link will be valid for 1 hour.
+Download at <%= @url %>. This link will be valid for 24 hours.

app/workers/concerns/dump_worker.rb

@@ -67,7 +67,7 @@ def emails
   end
 
   def send_email
-    mailer.perform_async(@project.id, medium.get_url, emails)
+    mailer.perform_async(@project.id, medium.get_url(get_expires: 24*60), emails)
   end
 
   def to_gzip

spec/models/medium_spec.rb

@@ -89,6 +89,11 @@
           .with(anything, be_a(HashWithIndifferentAccess))
         medium.put_url
       end
+
+      it 'should pass opts to MediaStorage' do
+        expect(MediaStorage).to receive(:put_path).with(anything, hash_including('foo' => 'bar'))
+        medium.put_url(foo: 'bar')
+      end
     end
 
     context "when externally linked" do
@@ -111,6 +116,11 @@
           .with(anything, be_a(HashWithIndifferentAccess))
         medium.get_url
       end
+
+      it 'should pass opts to MediaStorage' do
+        expect(MediaStorage).to receive(:get_path).with(anything, hash_including('foo' => 'bar'))
+        medium.get_url(foo: 'bar')
+      end
     end
 
     context "when externally linked" do