Run tests on some additional Ruby versions

.github/workflows/test.yml

@@ -0,0 +1,34 @@
+on:
+  - push
+  - pull_request
+
+jobs:
+  test:
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby:
+          - ruby-2.0
+          - ruby-2.1
+          # - ruby-2.2
+          - ruby-2.3
+          - ruby-2.4
+          - ruby-2.5
+          - ruby-2.6
+          - ruby-2.7
+          - ruby-3.0
+          - ruby-3.1
+          - ruby-3.2
+          - jruby-9.1
+          - jruby-9.2
+          - jruby-9.3
+          - jruby-9.4
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - run: ruby --version
+      - run: bundle exec rspec

.travis.yml

@@ -1,14 +0,0 @@
-sudo: false
-language: ruby
-
-rvm:
-  - 1.9.3
-  - 2.0.0
-  - 2.1.7
-  - 2.2.3
-  - jruby-19mode
-  - jruby-9
-  - rbx-2.5
-
-script:
-  - bundle exec rspec -c spec

CHANGELOG.md

@@ -1,12 +1,21 @@
+### 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
+- 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
+- Don't allow configure() to be called with unrecognized options
 
 ### 0.1.0 / 2014-12-13
 
-* Initial release
+- Initial release

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,12 @@
+Copyright 2014-2017 James Coglan
+
+Licensed under the Apache License, Version 2.0 (the "License"); you may not use
+this file except in compliance with the License. You may obtain a copy of the
+License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software distributed
+under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR
+CONDITIONS OF ANY KIND, either express or implied. See the License for the
+specific language governing permissions and limitations under the License.

README.md

@@ -1,4 +1,4 @@
-# permessage_deflate [![Build status](https://secure.travis-ci.org/faye/permessage-deflate-ruby.svg)](http://travis-ci.org/faye/permessage-deflate-ruby)
+# permessage_deflate
 
 Implements the
 [permessage-deflate](https://tools.ietf.org/html/draft-ietf-hybi-permessage-compression)
@@ -11,7 +11,7 @@ WebSocket protocol extension as a plugin for
 $ gem install permessage_deflate
 ```
 
-## Usage
+## Usage
 
 Add the plugin to your extensions:
 
@@ -44,47 +44,24 @@ the peer, and those that are negotiated as part of the protocol. The settings
 only affecting the compressor are described fully in the [Zlib
 documentation](http://ruby-doc.org/stdlib-2.1.0/libdoc/zlib/rdoc/Zlib/Deflate.html#method-c-new):
 
-* `:level`: sets the compression level, can be an integer from `0` to `9`, or
+- `:level`: sets the compression level, can be an integer from `0` to `9`, or
   one of the constants `Zlib::NO_COMPRESSION`, `Zlib::BEST_SPEED`,
   `Zlib::BEST_COMPRESSION`, or `Zlib::DEFAULT_COMPRESSION`
-* `:mem_level`: sets how much memory the compressor allocates, can be an integer
+- `:mem_level`: sets how much memory the compressor allocates, can be an integer
   from `1` to `9`, or one of the constants `Zlib::MAX_MEM_LEVEL`, or
   `Zlib::DEF_MEM_LEVEL`
-* `:strategy`: can be one of the constants `Zlib::FILTERED`,
+- `:strategy`: can be one of the constants `Zlib::FILTERED`,
   `Zlib::HUFFMAN_ONLY`, `Zlib::RLE`, `Zlib::FIXED`, or `Zlib::DEFAULT_STRATEGY`
 
 The other options relate to settings that are negotiated via the protocol and
 can be used to set the local session's behaviour and control that of the peer:
 
-* `:no_context_takeover`: if `true`, stops the session reusing a deflate context
+- `:no_context_takeover`: if `true`, stops the session reusing a deflate context
   between messages
-* `:request_no_context_takeover`: if `true`, makes the session tell the other
+- `:request_no_context_takeover`: if `true`, makes the session tell the other
   peer not to reuse a deflate context between messages
-* `:max_window_bits`: an integer from `8` to `15` inclusive that sets the
+- `:max_window_bits`: an integer from `8` to `15` inclusive that sets the
   maximum size of the session's sliding window; a lower window size will be used
   if requested by the peer
-* `:request_max_window_bits`: an integer from `8` to `15` inclusive to ask the
+- `: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-2015 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

@@ -21,7 +21,7 @@ class PermessageDeflate
   def self.validate_options(options, valid_keys)
     options.keys.each do |key|
       unless valid_keys.include?(key)
-        raise ConfigurationError, "Unrecognized option: #{key.inspect}"
+        raise ConfigurationError, "Unrecognized option: #{ key.inspect }"
       end
     end
   end
@@ -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/server_session.rb

@@ -19,21 +19,21 @@ class PermessageDeflate
     def generate_response
       response = {}
 
-      # https://tools.ietf.org/html/draft-ietf-hybi-permessage-compression#section-8.1.1.1
+      # 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/draft-ietf-hybi-permessage-compression#section-8.1.1.2
+      # 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/draft-ietf-hybi-permessage-compression#section-8.1.2.1
+      # 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
@@ -45,7 +45,7 @@ class PermessageDeflate
         response['server_max_window_bits'] = @own_window_bits
       end
 
-      # https://tools.ietf.org/html/draft-ietf-hybi-permessage-compression#section-8.1.2.2
+      # https://tools.ietf.org/html/rfc7692#section-7.1.2.2
 
       if client_max = @params['client_max_window_bits']
         client_max = MAX_WINDOW_BITS if client_max == true

lib/permessage_deflate/session.rb

@@ -8,6 +8,7 @@ class PermessageDeflate
       'client_max_window_bits'
     ]
 
+    MIN_WINDOW_BITS   = 9
     MAX_WINDOW_BITS   = 15
     VALID_WINDOW_BITS = [8, 9, 10, 11, 12, 13, 14, 15]
 
@@ -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,17 +1,17 @@
 Gem::Specification.new do |s|
-  s.name              = 'permessage_deflate'
-  s.version           = '0.1.2'
-  s.summary           = 'Per-message DEFLATE compression extension for WebSocket connections'
-  s.author            = 'James Coglan'
-  s.email             = 'jcoglan@gmail.com'
-  s.homepage          = 'http://github.com/faye/permessage-deflate-ruby'
-  s.license           = 'MIT'
+  s.name     = 'permessage_deflate'
+  s.version  = '0.1.4'
+  s.summary  = 'Per-message DEFLATE compression extension for WebSocket connections'
+  s.author   = 'James Coglan'
+  s.email    = 'jcoglan@gmail.com'
+  s.homepage = 'https://github.com/faye/permessage-deflate-ruby'
+  s.license  = 'Apache-2.0'
 
-  s.extra_rdoc_files  = %w[README.md]
-  s.rdoc_options      = %w[--main README.md --markup markdown]
-  s.require_paths     = %w[lib]
+  s.extra_rdoc_files = %w[README.md]
+  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
@@ -204,7 +204,7 @@ describe PermessageDeflate::ClientSession do
     before { options[:max_window_bits] = 20 }
 
     it "raises when generating the offer" do
-      expect { offer }.to raise_error
+      expect { offer }.to raise_error(PermessageDeflate::ConfigurationError)
     end
   end
 
@@ -286,7 +286,7 @@ describe PermessageDeflate::ClientSession do
     before { options[:request_max_window_bits] = 20 }
 
     it "raises when generating an offer" do
-      expect { offer }.to raise_error
+      expect { offer }.to raise_error(PermessageDeflate::ConfigurationError)
     end
   end