degzipper 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cfe03a119e7b59ea20518cb877c335abb717f1f6
-  data.tar.gz: 1d4121372cf7b74a828f344386aaa7a47b7e45dd
+  metadata.gz: 1151434f8833470a8de8ba07d73bd3430c2330e9
+  data.tar.gz: 51b24f05136a687187e2977a6bf7e589d118c836
 SHA512:
-  metadata.gz: bd6ed2d875f4fa97880e9bb3b53d84f8866775156d48c3d00ff7cfd12c0ed9ce8c566b62d8403c57dfb9e6ef902f22d0aaef3e17acc0cf06b634a9e992341ee6
-  data.tar.gz: af4d58065269ff00f4b2101840c234507611df8dd8c5389c8f3f4d65667b21c8a4936221c60ec8c05cd99adede95515dd29d8c24a09816eca6352fce8fdd5c95
+  metadata.gz: c7f93d6007d513925ab291f3757e502d7757d15126585116725d5ffa8e7b75c14f981d0802a2d1bdeb35d338525b425cbaf5b844b0c5d47f991822b4d631a21e
+  data.tar.gz: 19149bcb8ac05e5470c0f35b13a626ad474dfeb18cdd754dc9e31bd599ee1866592e7f9bf3299e452a52ced7b5980525f700a85285c13472b675e78578e45fc6

data/.rspec

@@ -0,0 +1 @@
+--color

data/Rakefile

@@ -1 +1,6 @@
 require "bundler/gem_tasks"
+require "rspec/core/rake_task"
+
+RSpec::Core::RakeTask.new(:spec)
+
+task default: :spec

data/degzipper.gemspec

@@ -20,4 +20,5 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
+  spec.add_development_dependency "rspec"
 end

data/lib/degzipper/middleware.rb

@@ -17,7 +17,7 @@ module Degzipper
         extracted = decode(env['rack.input'], env['HTTP_CONTENT_ENCODING'])
 
         env.delete('HTTP_CONTENT_ENCODING')
-        env['CONTENT_LENGTH'] = extracted.length
+        env['CONTENT_LENGTH'] = extracted.bytesize
         env['rack.input'] = StringIO.new(extracted)
       end
 

data/lib/degzipper/version.rb

@@ -1,3 +1,3 @@
 module Degzipper
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/degzipper_spec.rb

@@ -0,0 +1,80 @@
+require 'degzipper'
+require 'json'
+require 'rack'
+require 'zlib'
+
+RSpec.describe Degzipper::Middleware do
+  def gzip(content)
+    string_io = StringIO.new
+
+    gz = Zlib::GzipWriter.new(string_io)
+    gz.write content
+    gz.close
+
+    string_io.string
+  end
+
+  let(:middleware) do
+    Degzipper::Middleware.new(-> (env) do
+      req = Rack::Request.new(env)
+
+      body = JSON.dump(
+        body: req.body.read,
+        content_encoding: env['HTTP_CONTENT_ENCODING'],
+        length: req.content_length.to_i
+      )
+
+      [200, {}, [body]]
+    end)
+  end
+
+  it 'passes through a non-gzipped request body' do
+    _, _, body = middleware.call(Rack::MockRequest.env_for(
+      '/',
+      method: 'POST',
+      input: 'hello'
+    ))
+
+    parsed_body = JSON.parse(body.first)
+
+    expect(parsed_body).to eq(
+      'body' => 'hello',
+      'content_encoding' => nil,
+      'length' => 5
+    )
+  end
+
+  it 'extracts a gzipped request body' do
+    _, _, body = middleware.call(Rack::MockRequest.env_for(
+      '/',
+      method: 'POST',
+      input: gzip('hello'),
+      'HTTP_CONTENT_ENCODING' => 'gzip'
+    ))
+
+    parsed_body = JSON.parse(body.first)
+
+    expect(parsed_body).to eq(
+      'body' => 'hello',
+      'content_encoding' => nil,
+      'length' => 5
+    )
+  end
+
+  it 'sets the correct content length for UTF-8 content' do
+    _, _, body = middleware.call(Rack::MockRequest.env_for(
+      '/',
+      method: 'POST',
+      input: gzip('你好'),
+      'HTTP_CONTENT_ENCODING' => 'gzip'
+    ))
+
+    parsed_body = JSON.parse(body.first)
+
+    expect(parsed_body).to eq(
+      'body' => '你好',
+      'content_encoding' => nil,
+      'length' => 6
+    )
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: degzipper
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Andrew Hammond
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-01 00:00:00.000000000 Z
+date: 2015-05-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -39,6 +39,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Rack middleware to inflate incoming Gzip data from HTTP requests.
 email:
 - andrew@evertrue.com
@@ -47,6 +61,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -56,6 +71,7 @@ files:
 - lib/degzipper/middleware.rb
 - lib/degzipper/rails.rb
 - lib/degzipper/version.rb
+- spec/degzipper_spec.rb
 homepage: http://github.com/andrhamm/degzipper
 licenses:
 - MIT
@@ -80,4 +96,5 @@ rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Rack middleware to inflate incoming Gzip data from HTTP requests.
-test_files: []
+test_files:
+- spec/degzipper_spec.rb