rack-stripper 1.0.1 → 1.1.0

data/lib/rack/stripper.rb

@@ -2,14 +2,14 @@ require 'rack'
 
 module Rack
   class Stripper
-    VERSION = '1.0.1'
+    VERSION = '1.1.0'
 
     attr_accessor :add_xml_instruction
     attr_accessor :is_xml_response
 
     def initialize(app, options={})
       @app = app
-      self.add_xml_instruction = options[:add_xml_instruction] || false
+      @add_xml_instruction = options[:add_xml_instruction] || false
     end
 
     def call(env)
@@ -19,16 +19,22 @@ module Rack
     def _call(env)
       status, headers, response = @app.call(env)
 
+      if status != 200
+        return [status, headers, response]
+      end
+
       if headers.has_key?('Content-Type')
         self.is_xml_response = !headers['Content-Type'].match(/xml/).nil?
       end
 
       if response.respond_to?(:body)
-        stripped_body = process_body(response.body)
-        headers['Content-Length'] = stripped_body.bytesize if headers.has_key?('Content-Length')
-        response = Rack::Response.new(stripped_body, status, headers)
-        response.to_a
+        response.body = process_body(response.body)
+        if headers.has_key?('Content-Length')
+          headers['Content-Length'] = response.body.bytesize
+        end
       end
+
+      [status, headers, response]
     end
 
     protected
@@ -36,7 +42,7 @@ module Rack
     def process_body(body)
       body = body[0] if body.is_a?(Array)
       body.strip!
-      if self.add_xml_instruction && doesnt_have_xml_instruction_already?(body) && self.is_xml_response
+      if @add_xml_instruction && @is_xml_response && lacks_instruction?(body)
         body = "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n#{body}"
       end
       body
@@ -48,8 +54,8 @@ module Rack
     # @param [String] body The body of the text.
     #
     # @return [Boolean] true if the content doesn't have an XML instruction, false if it does.
-    def doesnt_have_xml_instruction_already?(body)
+    def lacks_instruction?(body)
       body.index('<?xml ').nil? rescue false
     end
   end
-end
+end

data/rack-stripper.gemspec

@@ -16,6 +16,7 @@ Gem::Specification.new do |gem|
   gem.version       = Rack::Stripper::VERSION
 
   gem.add_development_dependency 'rspec'
+  gem.add_development_dependency 'rake'
   gem.add_development_dependency 'debugger'
-  gem.add_dependency 'rack', '>= 1.0.0'
+  gem.add_runtime_dependency 'rack', '~> 1.4.5'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rack-stripper
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-27 00:00:00.000000000 Z
+date: 2013-03-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -27,6 +27,22 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: debugger
   requirement: !ruby/object:Gem::Requirement
@@ -48,17 +64,17 @@ dependencies:
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ! '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.4.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ! '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.4.5
 description: Trims whitespace from response bodies sent through rack.
 email:
 - ben@kree.gr
@@ -90,7 +106,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 705178727278721884
+      hash: 3041786719630630150
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -99,7 +115,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 705178727278721884
+      hash: 3041786719630630150
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23