Bump version to 0.1.4.

.travis.yml

@@ -1,10 +1,18 @@
+sudo: false
 language: ruby
 
 rvm:
   - 1.9.3
   - 2.0.0
-  - 2.1.3
+  - 2.1.10
+  - 2.2.7
+  - 2.3.4
+  - 2.4.1
   - jruby-19mode
-  - rbx-2.2
+  - jruby-9
 
-script: bundle exec rspec -c spec/
+before_install:
+  - '[[ "$(ruby --version)" == *"1.9.3"* ]] && gem update --system 2.4.8 || true'
+
+script:
+  - bundle exec rspec -c spec

CHANGELOG.md

@@ -1,3 +1,17 @@
+### 0.1.4 / 2017-09-10
+
+* Use `9` instead of `8` as the `windowBits` parameter to zlib, to deal with
+  restrictions introduced in zlib v1.2.9
+
+### 0.1.3 / 2016-05-20
+
+* Amend all warnings issued when running with -W2
+
+### 0.1.2 / 2015-11-06
+
+* The server does not send `server_max_window_bits` if the client does not ask
+  for it; this works around an issue in Firefox.
+
 ### 0.1.1 / 2014-12-18
 
 * Don't allow configure() to be called with unrecognized options

CODE_OF_CONDUCT.md

@@ -0,0 +1,4 @@
+# Code of Conduct
+
+All projects under the [Faye](https://github.com/faye) umbrella are covered by
+the [Code of Conduct](https://github.com/faye/code-of-conduct).

LICENSE.md

@@ -0,0 +1,21 @@
+# The MIT License
+
+Copyright (c) 2014-2017 James Coglan
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the 'Software'), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
+the Software, and to permit persons to whom the Software is furnished to do so,
+subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
+FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
+COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
+IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+

README.md

@@ -11,7 +11,7 @@ WebSocket protocol extension as a plugin for
 $ gem install permessage_deflate
 ```
 
-## Usage
+## Usage
 
 Add the plugin to your extensions:
 
@@ -65,27 +65,3 @@ can be used to set the local session's behaviour and control that of the peer:
   if requested by the peer
 * `:request_max_window_bits`: an integer from `8` to `15` inclusive to ask the
   other peer to use to set its maximum sliding window size, if supported
-
-## License
-
-(The MIT License)
-
-Copyright (c) 2014 James Coglan
-
-Permission is hereby granted, free of charge, to any person obtaining a copy of
-this software and associated documentation files (the 'Software'), to deal in
-the Software without restriction, including without limitation the rights to
-use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies
-of the Software, and to permit persons to whom the Software is furnished to do
-so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.

lib/permessage_deflate.rb

@@ -34,6 +34,8 @@ class PermessageDeflate
     define_method(:rsv3) { false }
 
     def configure(options)
+      @options ||= nil
+
       PermessageDeflate.validate_options(options, VALID_OPTIONS)
       options = (@options || {}).merge(options)
       PermessageDeflate.new(options)

lib/permessage_deflate/client_session.rb

@@ -59,12 +59,12 @@ class PermessageDeflate
 
       @own_context_takeover = !(@accept_no_context_takeover || params['client_no_context_takeover'])
       @own_window_bits = [
-        @accept_max_window_bits || DEFAULT_MAX_WINDOW_BITS,
-        params['client_max_window_bits'] || DEFAULT_MAX_WINDOW_BITS
+        @accept_max_window_bits || MAX_WINDOW_BITS,
+        params['client_max_window_bits'] || MAX_WINDOW_BITS
       ].min
 
       @peer_context_takeover = !params['server_no_context_takeover']
-      @peer_window_bits = params['server_max_window_bits'] || DEFAULT_MAX_WINDOW_BITS
+      @peer_window_bits = params['server_max_window_bits'] || MAX_WINDOW_BITS
 
       true
     end

lib/permessage_deflate/server_session.rb

@@ -17,42 +17,48 @@ class PermessageDeflate
     end
 
     def generate_response
-      params = {}
+      response = {}
 
-      # https://tools.ietf.org/html/draft-ietf-hybi-permessage-compression#section-8.1.1.1
-      if @accept_no_context_takeover or @params['server_no_context_takeover']
-        params['server_no_context_takeover'] = true
+      # https://tools.ietf.org/html/rfc7692#section-7.1.1.1
+
+      @own_context_takeover = !@accept_no_context_takeover &&
+                              !@params['server_no_context_takeover']
+
+      response['server_no_context_takeover'] = true unless @own_context_takeover
+
+      # https://tools.ietf.org/html/rfc7692#section-7.1.1.2
+
+      @peer_context_takeover = !@request_no_context_takeover &&
+                               !@params['client_no_context_takeover']
+
+      response['client_no_context_takeover'] = true unless @peer_context_takeover
+
+      # https://tools.ietf.org/html/rfc7692#section-7.1.2.1
+
+      @own_window_bits = [ @accept_max_window_bits || MAX_WINDOW_BITS,
+                           @params['server_max_window_bits'] || MAX_WINDOW_BITS
+                         ].min
+
+      # In violation of the spec, Firefox closes the connection if it does not
+      # send server_max_window_bits but the server includes this in its response
+      if @own_window_bits < MAX_WINDOW_BITS and @params['server_max_window_bits']
+        response['server_max_window_bits'] = @own_window_bits
       end
 
-      # https://tools.ietf.org/html/draft-ietf-hybi-permessage-compression#section-8.1.1.2
-      if @request_no_context_takeover or @params['client_no_context_takeover']
-        params['client_no_context_takeover'] = true
-      end
+      # https://tools.ietf.org/html/rfc7692#section-7.1.2.2
 
-      # https://tools.ietf.org/html/draft-ietf-hybi-permessage-compression#section-8.1.2.1
-      if @accept_max_window_bits or @params['server_max_window_bits']
-        accept_max = @accept_max_window_bits || DEFAULT_MAX_WINDOW_BITS
-        server_max = @params['server_max_window_bits'] || DEFAULT_MAX_WINDOW_BITS
-        params['server_max_window_bits'] = [accept_max, server_max].min
-      end
-
-      # https://tools.ietf.org/html/draft-ietf-hybi-permessage-compression#section-8.1.2.2
       if client_max = @params['client_max_window_bits']
-        if client_max == true
-          params['client_max_window_bits'] = @request_max_window_bits if @request_max_window_bits
-        else
-          request_max = @request_max_window_bits || DEFAULT_MAX_WINDOW_BITS
-          params['client_max_window_bits'] = [request_max, client_max].min
-        end
+        client_max = MAX_WINDOW_BITS if client_max == true
+        @peer_window_bits = [@request_max_window_bits || MAX_WINDOW_BITS, client_max].min
+      else
+        @peer_window_bits = MAX_WINDOW_BITS
       end
 
-      @own_context_takeover = !params['server_no_context_takeover']
-      @own_window_bits = params['server_max_window_bits'] || DEFAULT_MAX_WINDOW_BITS
+      if @peer_window_bits < MAX_WINDOW_BITS
+        response['client_max_window_bits'] = @peer_window_bits
+      end
 
-      @peer_context_takeover = !params['client_no_context_takeover']
-      @peer_window_bits = params['client_max_window_bits'] || DEFAULT_MAX_WINDOW_BITS
-
-      params
+      response
     end
 
   end

lib/permessage_deflate/session.rb

@@ -8,7 +8,8 @@ class PermessageDeflate
       'client_max_window_bits'
     ]
 
-    DEFAULT_MAX_WINDOW_BITS = 15
+    MIN_WINDOW_BITS   = 9
+    MAX_WINDOW_BITS   = 15
     VALID_WINDOW_BITS = [8, 9, 10, 11, 12, 13, 14, 15]
 
     def self.valid_params?(params)
@@ -39,6 +40,8 @@ class PermessageDeflate
       @accept_max_window_bits      = options.fetch(:max_window_bits, nil)
       @request_no_context_takeover = options.fetch(:request_no_context_takeover, false)
       @request_max_window_bits     = options.fetch(:request_max_window_bits, nil)
+
+      @deflate = @inflate = nil
     end
 
     def process_incoming_message(message)
@@ -81,14 +84,20 @@ class PermessageDeflate
 
     def get_inflate
       return @inflate if @inflate
-      inflate = Zlib::Inflate.new(-@peer_window_bits)
+
+      window_bits = [@peer_window_bits, MIN_WINDOW_BITS].max
+      inflate     = Zlib::Inflate.new(-window_bits)
+
       @inflate = inflate if @peer_context_takeover
       inflate
     end
 
     def get_deflate
       return @deflate if @deflate
-      deflate = Zlib::Deflate.new(@level, -@own_window_bits, @mem_level, @strategy)
+
+      window_bits = [@own_window_bits, MIN_WINDOW_BITS].max
+      deflate     = Zlib::Deflate.new(@level, -window_bits, @mem_level, @strategy)
+
       @deflate = deflate if @own_context_takeover
       deflate
     end

permessage_deflate.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name              = 'permessage_deflate'
-  s.version           = '0.1.1'
+  s.version           = '0.1.4'
   s.summary           = 'Per-message DEFLATE compression extension for WebSocket connections'
   s.author            = 'James Coglan'
   s.email             = 'jcoglan@gmail.com'
@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   s.rdoc_options      = %w[--main README.md --markup markdown]
   s.require_paths     = %w[lib]
 
-  s.files = %w[README.md CHANGELOG.md] + Dir.glob('lib/**/*.rb')
+  s.files = %w[CHANGELOG.md LICENSE.md README.md] + Dir.glob('lib/**/*.rb')
 
   s.add_development_dependency 'rspec'
 end

spec/permessage_deflate/client_session_spec.rb

@@ -90,9 +90,9 @@ describe PermessageDeflate::ClientSession do
         expect(activate).to be true
       end
 
-      it "uses context takeover and 8 window bits for inflating incoming messages" do
+      it "uses context takeover and 9 window bits for inflating incoming messages" do
         activate
-        expect(Zlib::Inflate).to receive(:new).with(-8).exactly(1).and_return(inflate)
+        expect(Zlib::Inflate).to receive(:new).with(-9).exactly(1).and_return(inflate)
         process_incoming_message
         process_incoming_message
       end
@@ -113,9 +113,9 @@ describe PermessageDeflate::ClientSession do
         expect(activate).to be true
       end
 
-      it "uses context takeover and 8 window bits for deflating outgoing messages" do
+      it "uses context takeover and 9 window bits for deflating outgoing messages" do
         activate
-        expect(Zlib::Deflate).to receive(:new).with(level, -8, mem_level, strategy).exactly(1).and_return(deflate)
+        expect(Zlib::Deflate).to receive(:new).with(level, -9, mem_level, strategy).exactly(1).and_return(deflate)
         process_outgoing_message
         process_outgoing_message
       end
@@ -191,9 +191,9 @@ describe PermessageDeflate::ClientSession do
         expect(activate).to be true
       end
 
-      it "uses context takeover and 8 window bits for deflating outgoing messages" do
+      it "uses context takeover and 9 window bits for deflating outgoing messages" do
         activate
-        expect(Zlib::Deflate).to receive(:new).with(level, -8, mem_level, strategy).exactly(1).and_return(deflate)
+        expect(Zlib::Deflate).to receive(:new).with(level, -9, mem_level, strategy).exactly(1).and_return(deflate)
         process_outgoing_message
         process_outgoing_message
       end

spec/permessage_deflate/server_session_spec.rb

@@ -163,8 +163,8 @@ describe PermessageDeflate::ServerSession do
     before { options[:max_window_bits] = 12 }
 
     describe "with an empty offer" do
-      it "includes server_max_window_bits in the response" do
-        expect(response).to eq("server_max_window_bits" => 12)
+      it "does not include server_max_window_bits in the response" do
+        expect(response).to eq({})
       end
 
       it "uses context takeover and 12 window bits for deflating outgoing messages" do