From aedb04a90eb5d12ce6ef2afbedeb359ca78b715a Mon Sep 17 00:00:00 2001
From: Marcin Kulik <m@ku1ik.com>
Date: Sun, 10 Apr 2016 15:12:35 +0000
Subject: [PATCH] Fix json downloading for Safari

---
 app/serializers/asciicast_serializer.rb | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/app/serializers/asciicast_serializer.rb b/app/serializers/asciicast_serializer.rb
index f448ae1..34de05e 100644
--- a/app/serializers/asciicast_serializer.rb
+++ b/app/serializers/asciicast_serializer.rb
@@ -11,7 +11,12 @@ class AsciicastSerializer < ActiveModel::Serializer
     if v0_url?
       object.stdout_frames_url
     else
-      asciicast_path(object, format: :json)
+      url = object.data.url
+      if url.starts_with?("/")
+        asciicast_path(object, format: :json) # download URL for local carrierwave store
+      else
+        url # direct signed S3 URL (because XHR CORS redirect doesn't work under Safari)
+      end
     end
   end