http 4.1.1 → 4.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1762fba28fba17df3f1257ea7b1fab8682e21d5348b41e9a22aba977f23061d
-  data.tar.gz: af1584f93c3ee068274fadf006588f28bc258a797fa6ac03b506c49d43d10075
+  metadata.gz: ddb21214e710b5499c1d1074df10cb2467733d476996ff4b44ef476aad638d1a
+  data.tar.gz: a7e2ef9bb78b1b1b0bc2e06d8421d3d14bc4c4fe9ec5cedcafdcf1d4951978ee
 SHA512:
-  metadata.gz: 34b2c514ffa2833bb2e276503f37dead1de43e987928031597f30bf431ac1293db2ec2f72aec6afbfc12e9ece16839db5e27536df8830a0d7abfff59b8727bfc
-  data.tar.gz: cb5d5e41172fe3d9bf0619f1b359ff01246738c698a759f4e78e700b80c3174a0a223810ba03468a7c1300491630442698fd549e87f8a46b3bfc7f159acf9dac
+  metadata.gz: cba83eabbf6eed0eef8fcbd15da030a40d65c5ea8eceb1169cfdf2e114dbfb472762e673ddfd4552060b59b643d11bad4b70cbbda73dac9aceb05f15b772301c
+  data.tar.gz: c041c6ced64a2bdd994444975f3877d615be07245d91fb6825f4f8c9302b2c15aa1b98205b3401f6ebd77689d9831fb8982df15ba45478fd91703531328ea2f3

data/.rubocop.yml

@@ -20,7 +20,8 @@ Metrics/AbcSize:
 
 Metrics/BlockLength:
   Exclude:
-    - spec/**/*
+    - "spec/**/*"
+    - "**/*.gemspec"
 
 Metrics/BlockNesting:
   Max: 2

data/.travis.yml

@@ -17,18 +17,19 @@ env: JRUBY_OPTS="$JRUBY_OPTS --debug"
 
 rvm:
   # Include JRuby first because it takes the longest
-  - jruby-9.1.16.0
+  - jruby-9.2.5.0
   - 2.3
   - 2.4
   - 2.5
+  - 2.6
 
 matrix:
   fast_finish: true
   include:
   # Only run RuboCop and Yardstick metrics on the latest Ruby
-  - rvm: 2.5
+  - rvm: 2.6
     env: SUITE="rubocop"
-  - rvm: 2.5
+  - rvm: 2.6
     env: SUITE="yardstick"
 
 branches:

data/CHANGES.md

@@ -1,3 +1,10 @@
+## 4.2.0 (2019-10-22)
+
+* Backport [#489](https://github.com/httprb/http/pull/489)
+  Fix HTTP parser.
+  ([@ixti], [@fxposter])
+
+
 ## 4.1.1 (2019-03-12)
 
 * Add `HTTP::Headers::ACCEPT_ENCODING` constant.
@@ -771,4 +778,5 @@ end
 [@tycoon]: https://github.com/tycooon
 [@paul]: https://github.com/paul
 [@RickCSong]: https://github.com/RickCSong
+[@fxposter]: https://github.com/fxposter
 [@mamoonraja]: https://github.com/mamoonraja

data/README.md

@@ -4,7 +4,7 @@
 [![Build Status](https://secure.travis-ci.org/httprb/http.svg?branch=4-x-stable)](https://travis-ci.org/httprb/http)
 [![Code Climate](https://codeclimate.com/github/httprb/http.svg?branch=4-x-stable)](https://codeclimate.com/github/httprb/http)
 [![Coverage Status](https://coveralls.io/repos/httprb/http/badge.svg?branch=4-x-stable)](https://coveralls.io/r/httprb/http)
-[![MIT licensed](https://img.shields.io/badge/license-MIT-blue.svg)](https://github.com/httprb/http/blob/master/LICENSE.txt)
+[![MIT licensed](https://img.shields.io/badge/license-MIT-blue.svg)](https://github.com/httprb/http/blob/4-x-stable/LICENSE.txt)
 
 [Documentation]
 
@@ -167,7 +167,8 @@ versions:
 * Ruby 2.3.x
 * Ruby 2.4.x
 * Ruby 2.5.x
-* JRuby 9.1.x.x
+* Ruby 2.6.x
+* JRuby 9.2.x.x
 
 If something doesn't work on one of these versions, it's a bug.
 
@@ -197,5 +198,5 @@ dropped.
 
 ## Copyright
 
-Copyright (c) 2011-2018 Tony Arcieri, Alexey V. Zapparov, Erik Michaels-Ober, Zachary Anker.
+Copyright (c) 2011-2019 Tony Arcieri, Alexey V. Zapparov, Erik Michaels-Ober, Zachary Anker.
 See LICENSE.txt for further details.

data/http.gemspec

@@ -27,10 +27,10 @@ Gem::Specification.new do |gem|
 
   gem.required_ruby_version = ">= 2.3"
 
-  gem.add_runtime_dependency "http_parser.rb", "~> 0.6.0"
-  gem.add_runtime_dependency "http-form_data", "~> 2.0"
-  gem.add_runtime_dependency "http-cookie",    "~> 1.0"
   gem.add_runtime_dependency "addressable",    "~> 2.3"
+  gem.add_runtime_dependency "http-cookie",    "~> 1.0"
+  gem.add_runtime_dependency "http-form_data", "~> 2.0"
+  gem.add_runtime_dependency "http-parser",    "~> 1.2.0"
 
   gem.add_development_dependency "bundler", "~> 2.0"
 

data/lib/http.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "http/parser"
-
 require "http/errors"
 require "http/timeout/null"
 require "http/timeout/per_operation"

data/lib/http/response/parser.rb

@@ -1,41 +1,66 @@
 # frozen_string_literal: true
 
+require "http-parser"
+
 module HTTP
   class Response
+    # @api private
+    #
+    # NOTE(ixti): This class is a subject of future refactoring, thus don't
+    #   expect this class API to be stable until this message disappears and
+    #   class is not marked as private anymore.
     class Parser
       attr_reader :headers
 
       def initialize
-        @parser = HTTP::Parser.new(self)
+        @state  = HttpParser::Parser.new_instance { |i| i.type = :response }
+        @parser = HttpParser::Parser.new(self)
+
         reset
       end
 
+      # @return [self]
       def add(data)
-        @parser << data
+        # XXX(ixti): API doc of HttpParser::Parser is misleading, it says that
+        #   it returns boolean true if data was parsed successfully, but instead
+        #   it's response tells if there was an error; So when it's `true` that
+        #   means parse failed, and `false` means parse was successful.
+        #   case of success.
+        return self unless @parser.parse(@state, data)
+
+        raise IOError, "Could not parse data"
       end
       alias << add
 
       def headers?
-        !!@headers
+        @finished[:headers]
       end
 
       def http_version
-        @parser.http_version.join(".")
+        @state.http_version
       end
 
       def status_code
-        @parser.status_code
+        @state.http_status
       end
 
       #
       # HTTP::Parser callbacks
       #
 
-      def on_headers_complete(headers)
-        @headers = headers
+      def on_header_field(_response, field)
+        @field = field
+      end
+
+      def on_header_value(_response, value)
+        @headers.add(@field, value) if @field
+      end
+
+      def on_headers_complete(_reposse)
+        @finished[:headers] = true
       end
 
-      def on_body(chunk)
+      def on_body(_response, chunk)
         if @chunk
           @chunk << chunk
         else
@@ -57,20 +82,21 @@ module HTTP
         chunk
       end
 
-      def on_message_complete
-        @finished = true
+      def on_message_complete(_response)
+        @finished[:message] = true
       end
 
       def reset
-        @parser.reset!
+        @state.reset!
 
-        @finished = false
-        @headers  = nil
+        @finished = Hash.new(false)
+        @headers  = HTTP::Headers.new
+        @field    = nil
         @chunk    = nil
       end
 
       def finished?
-        @finished
+        @finished[:message]
       end
     end
   end

data/lib/http/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HTTP
-  VERSION = "4.1.1"
+  VERSION = "4.2.0"
 end

data/spec/regression_specs.rb

@@ -14,4 +14,11 @@ RSpec.describe "Regression testing" do
       expect { HTTP.get(google_uri).to_s }.not_to raise_error
     end
   end
+
+  describe "#422" do
+    it "reads body when 200 OK response contains Upgrade header" do
+      res = HTTP.get("https://httpbin.org/response-headers?Upgrade=h2,h2c")
+      expect(res.parse(:json)).to include("Upgrade" => "h2,h2c")
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: http
 version: !ruby/object:Gem::Version
-  version: 4.1.1
+  version: 4.2.0
 platform: ruby
 authors:
 - Tony Arcieri
@@ -11,64 +11,64 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-12 00:00:00.000000000 Z
+date: 2019-10-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: http_parser.rb
+  name: addressable
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: '2.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: '2.3'
 - !ruby/object:Gem::Dependency
-  name: http-form_data
+  name: http-cookie
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: http-cookie
+  name: http-form_data
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: addressable
+  name: http-parser
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: 1.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: 1.2.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -186,7 +186,7 @@ metadata:
   source_code_uri: https://github.com/httprb/http
   wiki_uri: https://github.com/httprb/http/wiki
   bug_tracker_uri: https://github.com/httprb/http/issues
-  changelog_uri: https://github.com/httprb/http/blob/v4.1.1/CHANGES.md
+  changelog_uri: https://github.com/httprb/http/blob/v4.2.0/CHANGES.md
 post_install_message: 
 rdoc_options: []
 require_paths: