rack-app 2.0.0 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6883859ca0d72bc8fac97ac3b745a225bdbede0e
-  data.tar.gz: d53e14b3621d6290af628d6e11880629728eb206
+  metadata.gz: fcb9bbf91b7776d8c02a6c59f386ddb5893e68e9
+  data.tar.gz: a28f17db847b1242e62407da66beebe838ff7543
 SHA512:
-  metadata.gz: 80e9b3a47d3158c15837ca297e0a87f244859beae87ffbcedf59fc9a42987c19a63e7a2eab0c3c4b906bd5e66ed959b6461deb475dddca8e4772765211bfd587
-  data.tar.gz: da9cded93beb9b94645842ea4022f67b68633f0ee0c99355a53e217c6a79460ac375fbe43e3d73cbb63f5d082918b86594f5b64d9bdb30c8ec789096e0d336cb
+  metadata.gz: 9e291f4fda263c63f7bba524094789f8ae213cebe6c908453ed39fe8534b31a81d517539d15d0a01fa621213a3d33661ead8da980ca4aefb2d80cb69253b805e
+  data.tar.gz: 2a70f922341c3bda2209d1728d3e8021a5cd514177f03b4fd3454b6dce67e9d002595325d9e432f11b57063efc563cc6654306bda34f25af14ce74071ec95696

data/README.md

@@ -79,6 +79,8 @@ Yes, in fact it's already powering heroku hosted micro-services.
 * made with performance in mind so your app don't lose time by your framework
 * per endpoint middleware definitions
   * you can define middleware stack before endpoints and it will only applied to them, similar like protected method workflow
+* File Upload and file download in a efficient and elegant way with minimal memory consuming
+  * note that this is not only memory friendly way pure rack solution, but also 2x faster than the usualy solution which includes buffering in memory
 
 ## Usage
 

data/VERSION

@@ -1 +1 @@
-2.0.0
+2.1.0

data/lib/rack/app/instance_methods.rb

@@ -1,45 +1,13 @@
 module Rack::App::InstanceMethods
 
-  attr_writer :request, :response
-
-  def params
-    @__params__ ||= Rack::App::Params.new(request.env).to_hash
-  end
-
-  def request
-    @request || raise("request object is not set for #{self.class}")
-  end
-
-  def response
-    @response || raise("response object is not set for #{self.class}")
-  end
-
-  def payload
-    @__payload__ ||= lambda {
-      return nil unless @request.body.respond_to?(:gets)
-
-      payload = ''
-      while chunk = @request.body.gets
-        payload << chunk
-      end
-      @request.body.rewind
-
-      return payload
-    }.call
-  end
-
-  def redirect_to(url)
-    url = "#{url}?#{request.env['QUERY_STRING']}" unless request.env['QUERY_STRING'].empty?
-    response.status = 301
-    response.headers.merge!({'Location' => url})
-    'See Ya!'
-  end
-
-  def serve_file(file_path)
-    raw_rack_resp = Rack::App::FileServer.serve_file(request.env, file_path)
-    response.headers.merge!(raw_rack_resp[1])
-    response.body = raw_rack_resp.last
-    return nil
-  end
+  require 'rack/app/instance_methods/core'
+  require 'rack/app/instance_methods/http_control'
+  require 'rack/app/instance_methods/payload'
+  require 'rack/app/instance_methods/upload'
+
+  include Rack::App::InstanceMethods::Core
+  include Rack::App::InstanceMethods::HttpControl
+  include Rack::App::InstanceMethods::Payload
+  include Rack::App::InstanceMethods::Upload
 
 end

data/lib/rack/app/instance_methods/core.rb

@@ -0,0 +1,18 @@
+module Rack::App::InstanceMethods::Core
+
+  attr_writer :request, :response
+
+  def params
+    @__params__ ||= Rack::App::Params.new(request.env).to_hash
+  end
+
+  def request
+    @request || raise("request object is not set for #{self.class}")
+  end
+
+  def response
+    @response || raise("response object is not set for #{self.class}")
+  end
+
+end
+

data/lib/rack/app/instance_methods/http_control.rb

@@ -0,0 +1,11 @@
+module Rack::App::InstanceMethods::HttpControl
+
+  def redirect_to(url)
+    url = "#{url}?#{request.env['QUERY_STRING']}" unless request.env['QUERY_STRING'].empty?
+    response.status = 301
+    response.headers.merge!({'Location' => url})
+    'See Ya!'
+  end
+
+end
+

data/lib/rack/app/instance_methods/payload.rb

@@ -0,0 +1,29 @@
+module Rack::App::InstanceMethods::Payload
+
+  def payload_stream(&block)
+    return nil unless @request.body.respond_to?(:gets)
+    while chunk = request.body.gets
+      block.call(chunk)
+    end
+    request.body.rewind
+    nil
+  end
+
+  def payload
+    @__payload__ ||= lambda {
+
+      payload = ''
+      payload_stream { |chunk| payload << chunk }
+      return payload
+
+    }.call
+  end
+
+  def payload_to_file(file_path, file_mod='w')
+    File.open(file_path, file_mod) do |file|
+      payload_stream { |chunk| file.print(chunk) }
+    end
+  end
+
+end
+

data/lib/rack/app/instance_methods/upload.rb

@@ -0,0 +1,11 @@
+module Rack::App::InstanceMethods::Upload
+
+  def serve_file(file_path)
+    raw_rack_resp = Rack::App::FileServer.serve_file(request.env, file_path)
+    response.headers.merge!(raw_rack_resp[1])
+    response.body = raw_rack_resp.last
+    return nil
+  end
+
+end
+

data/lib/rack/app/test.rb

@@ -58,8 +58,23 @@ module Rack::App::Test
     properties = format_properties(raw_properties)
     additional_headers = properties[:headers].reduce({}) { |m, (k, v)| m.merge("HTTP_#{k.to_s.gsub('-', '_').upcase}" => v.to_s) }
 
+
     payload = raw_properties.delete(:payload)
-    additional_headers["rack.input"]= ::Rack::Lint::InputWrapper.new(StringIO.new(payload.to_s)) if payload.is_a?(String)
+
+    io = case payload
+
+           when IO
+             payload
+
+           when String
+             StringIO.new(payload.to_s)
+
+           else
+             StringIO.new('')
+
+         end
+
+    additional_headers["rack.input"]= ::Rack::Lint::InputWrapper.new(io)
 
     {
         "REMOTE_ADDR" => "192.168.56.1",

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rack-app
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.0
 platform: ruby
 authors:
 - Adam Luzsi
@@ -95,6 +95,10 @@ files:
 - lib/rack/app/error_handler.rb
 - lib/rack/app/file_server.rb
 - lib/rack/app/instance_methods.rb
+- lib/rack/app/instance_methods/core.rb
+- lib/rack/app/instance_methods/http_control.rb
+- lib/rack/app/instance_methods/payload.rb
+- lib/rack/app/instance_methods/upload.rb
 - lib/rack/app/params.rb
 - lib/rack/app/request_configurator.rb
 - lib/rack/app/router.rb