Support download acceleration using X-Sendfile

b1bd0d22 · Kamil Trzcinski · bce3d33c · b1bd0d22 · b1bd0d22
--- a/lib/api/helpers.rb
+++ b/lib/api/helpers.rb
@@ -293,7 +293,15 @@ def present_file!(path, filename, content_type = 'application/octet-stream')
      header['Content-Disposition'] = "attachment; filename=#{filename}"
      header['Content-Transfer-Encoding'] = 'binary'
      content_type content_type
-      file FileStreamer.new(path)
+
+      # Support download acceleration
+      case headers['X-Sendfile-Type']
+      when 'X-Sendfile'
+        header['X-Sendfile'] = path
+        body
+      else
+        file FileStreamer.new(path)
+      end
    end

    private

--- a/lib/ci/api/builds.rb
+++ b/lib/ci/api/builds.rb
@@ -95,7 +95,6 @@ class Builds < Grape::API
            not_found!
          end

-          # TODO: this is slow, because it doesn't support Rack::Sendfile
          present_file!(build.artifact_file.path, build.artifact_file.filename)
        end