protocol-http1 0.12.0 → 0.14.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f22aa54b84d5e367d855540a30f40103e384c2bed44d023cb14fa0be4024455b
-  data.tar.gz: ea92287d17970faf97174324907815798664be67316cd790af6fe7522263d586
+  metadata.gz: 884c811e792417717ddccec3f8369607dcc46ede6951557c567a08b3fa7d434e
+  data.tar.gz: 262f7ef658565c295964266c1eef6ff78464d39f7b7fa40bee7f44032d839570
 SHA512:
-  metadata.gz: f44d2e720314924e6b57c2f6e3e5f288ae8e4a5487d5730b9a343b959c0b28397594e8283e0180b70ebb5b75ccfab644b2bb609b0143bb2886c5d33d0b9dbdd5
-  data.tar.gz: c6afa65e6541ba2c0d8186c495c29f2fd3549b4a8e17cd91c379c409c8251e4146e10d82fc9835c761c040c4c48ea47934736f94ed9ee90b23716e2ee98b6c5c
+  metadata.gz: 6c991118f30f922938caa5a83f05d91f9f627c0fb5cdf6525aa08e7c559340514b571c636ae7e4048b951848fc2f4d481388901cfd6b3b42c0d0312aa00a51bf
+  data.tar.gz: 5f2e2cdb7b248fc83bd73d43b1e39102bd96663185994758b09d512adafabd59368e344ed38f472170bbac960a9d2386ad9d9e32ec01bb1fb05d0107f5bf509e

data/lib/protocol/http1/body/chunked.rb

@@ -26,7 +26,6 @@ module Protocol
 	module HTTP1
 		module Body
 			class Chunked < HTTP::Body::Readable
-				TRAILERS = 'trailers'
 				CRLF = "\r\n"
 				
 				def initialize(stream, headers)
@@ -57,12 +56,13 @@ module Protocol
 				def read
 					return nil if @finished
 					
+					# It is possible this line contains chunk extension, so we use `to_i` to only consider the initial integral part:
 					length = read_line.to_i(16)
 					
 					if length == 0
 						@finished = true
 						
-						read_trailers
+						read_trailer
 						
 						return nil
 					end
@@ -85,13 +85,17 @@ module Protocol
 				
 				private
 				
-				def read_line
+				def read_line?
 					@stream.gets(CRLF, chomp: true)
 				end
 				
-				def read_trailers
-					while line = read_line
-						# Empty line indicates end of headers:
+				def read_line
+					read_line? or raise EOFError
+				end
+				
+				def read_trailer
+					while line = read_line?
+						# Empty line indicates end of trailer:
 						break if line.empty?
 						
 						if match = line.match(HEADER)

data/lib/protocol/http1/connection.rb

@@ -281,14 +281,17 @@ module Protocol
 			
 			def write_fixed_length_body(body, length, head)
 				@stream.write("content-length: #{length}\r\n\r\n")
-				@stream.flush
 				
 				if head
+					@stream.flush
+					
 					body.close
 					
 					return
 				end
 				
+				@stream.flush unless body.ready?
+				
 				chunk_length = 0
 				body.each do |chunk|
 					chunk_length += chunk.bytesize
@@ -298,6 +301,7 @@ module Protocol
 					end
 					
 					@stream.write(chunk)
+					@stream.flush unless body.ready?
 				end
 				
 				@stream.flush
@@ -307,28 +311,32 @@ module Protocol
 				end
 			end
 			
-			def write_chunked_body(body, head, trailers = nil)
+			def write_chunked_body(body, head, trailer = nil)
 				@stream.write("transfer-encoding: chunked\r\n\r\n")
-				@stream.flush
 				
 				if head
+					@stream.flush
+					
 					body.close
 					
 					return
 				end
 				
+				@stream.flush unless body.ready?
+				
 				body.each do |chunk|
 					next if chunk.size == 0
 					
 					@stream.write("#{chunk.bytesize.to_s(16).upcase}\r\n")
 					@stream.write(chunk)
 					@stream.write(CRLF)
-					@stream.flush
+					
+					@stream.flush unless body.ready?
 				end
 				
-				if trailers
+				if trailer
 					@stream.write("0\r\n")
-					write_headers(trailers)
+					write_headers(trailer)
 					@stream.write("\r\n")
 				else
 					@stream.write("0\r\n\r\n")
@@ -342,25 +350,26 @@ module Protocol
 				@persistent = false
 				
 				@stream.write("\r\n")
-				@stream.flush
+				@stream.flush unless body.ready?
 				
 				if head
 					body.close
 				else
 					body.each do |chunk|
 						@stream.write(chunk)
-						@stream.flush
+						
+						@stream.flush unless body.ready?
 					end
 				end
 				
 				@stream.close_write
 			end
 			
-			def write_body(version, body, head = false, trailers = nil)
+			def write_body(version, body, head = false, trailer = nil)
 				if body.nil?
 					write_connection_header(version)
 					write_empty_body(body)
-				elsif length = body.length and (VERSION != HTTP11 && trailers.nil?)
+				elsif length = body.length and trailer.nil?
 					write_connection_header(version)
 					write_fixed_length_body(body, length, head)
 				elsif body.empty?
@@ -370,7 +379,7 @@ module Protocol
 				elsif @persistent and version == HTTP11
 					write_connection_header(version)
 					# We specifically ensure that non-persistent connections do not use chunked response, so that hijacking works as expected.
-					write_chunked_body(body, head, trailers)
+					write_chunked_body(body, head, trailer)
 				else
 					@persistent = false
 					write_connection_header(version)

data/lib/protocol/http1/version.rb

@@ -22,6 +22,6 @@
 
 module Protocol
 	module HTTP1
-		VERSION = "0.12.0"
+		VERSION = "0.14.1"
 	end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: protocol-http1
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.14.1
 platform: ruby
 authors:
 - Samuel Williams
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-08 00:00:00.000000000 Z
+date: 2021-05-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: protocol-http
@@ -16,28 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.22'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
-- !ruby/object:Gem::Dependency
-  name: covered
-  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'
+        version: '0.22'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -53,7 +39,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: bake-bundler
+  name: covered
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: rspec-memory
+  name: rspec-files
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -95,7 +81,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: rspec-files
+  name: rspec-memory
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -108,20 +94,12 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
-description: 
+description:
 email:
-- samuel.williams@oriontransfer.co.nz
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".editorconfig"
-- ".gitignore"
-- ".rspec"
-- ".travis.yml"
-- Gemfile
-- README.md
-- examples/http1/request.rb
 - lib/protocol/http1.rb
 - lib/protocol/http1/body/chunked.rb
 - lib/protocol/http1/body/fixed.rb
@@ -130,18 +108,17 @@ files:
 - lib/protocol/http1/error.rb
 - lib/protocol/http1/reason.rb
 - lib/protocol/http1/version.rb
-- protocol-http1.gemspec
 homepage: https://github.com/socketry/protocol-http1
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - "~>"
+  - - ">="
     - !ruby/object:Gem::Version
       version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
@@ -150,8 +127,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: A low level implementation of the HTTP/1 protocol.
 test_files: []

data/.editorconfig

@@ -1,6 +0,0 @@
-root = true
-
-[*]
-indent_style = tab
-indent_size = 2
-

data/.gitignore

@@ -1,13 +0,0 @@
-/.bundle/
-/.yardoc
-/_yardoc/
-/coverage/
-/doc/
-/pkg/
-/spec/reports/
-/tmp/
-
-# rspec failure tracking
-.rspec_status
-Gemfile.lock
-.covered.db

data/.rspec

@@ -1,3 +0,0 @@
---format documentation
---warnings
---require spec_helper

data/.travis.yml

@@ -1,21 +0,0 @@
-language: ruby
-dist: xenial
-cache: bundler
-
-script: bundle exec rspec
-
-matrix:
-  include:
-    - rvm: 2.5
-    - rvm: 2.6
-    - rvm: 2.7
-    - rvm: 2.6
-      env: COVERAGE=PartialSummary,Coveralls
-    - rvm: truffleruby
-    - rvm: jruby-head
-      env: JRUBY_OPTS="--debug -X+O"
-    - rvm: ruby-head
-  allow_failures:
-    - rvm: truffleruby
-    - rvm: ruby-head
-    - rvm: jruby-head

data/Gemfile

@@ -1,6 +0,0 @@
-# frozen_string_literal: true
-
-source "https://rubygems.org"
-
-# Specify your gem's dependencies in protocol-http1.gemspec
-gemspec

data/README.md

@@ -1,92 +0,0 @@
-# Protocol::HTTP1
-
-Provides a low-level implementation of the HTTP/1 protocol.
-
-[![Build Status](https://travis-ci.com/socketry/protocol-http1.svg?branch=master)](https://travis-ci.com/socketry/protocol-http1)
-
-## Installation
-
-Add this line to your application's Gemfile:
-
-```ruby
-gem 'protocol-http1'
-```
-
-And then execute:
-
-	$ bundle
-
-Or install it yourself as:
-
-	$ gem install protocol-http1
-
-## Usage
-
-Here is a basic HTTP/1.1 client:
-
-```ruby
-require 'async'
-require 'async/io/stream'
-require 'async/http/endpoint'
-require 'protocol/http1/connection'
-
-Async do
-	endpoint = Async::HTTP::Endpoint.parse("https://www.google.com/search?q=kittens", alpn_protocols: ["http/1.1"])
-	
-	peer = endpoint.connect
-	
-	puts "Connected to #{peer} #{peer.remote_address.inspect}"
-	
-	# IO Buffering...
-	stream = Async::IO::Stream.new(peer)
-	client = Protocol::HTTP1::Connection.new(stream)
-	
-	def client.read_line
-		@stream.read_until(Protocol::HTTP1::Connection::CRLF) or raise EOFError
-	end
-	
-	puts "Writing request..."
-	client.write_request("www.google.com", "GET", "/search?q=kittens", "HTTP/1.1", [["Accept", "*/*"]])
-	client.write_body(nil)
-	
-	puts "Reading response..."
-	response = client.read_response("GET")
-	
-	puts "Got response: #{response.inspect}"
-	
-	puts "Closing client..."
-	client.close
-end
-```
-
-## Contributing
-
-1. Fork it
-2. Create your feature branch (`git checkout -b my-new-feature`)
-3. Commit your changes (`git commit -am 'Add some feature'`)
-4. Push to the branch (`git push origin my-new-feature`)
-5. Create new Pull Request
-
-## License
-
-Released under the MIT license.
-
-Copyright, 2019, by [Samuel G. D. Williams](http://www.codeotaku.com/samuel-williams).  
-
-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.

data/examples/http1/request.rb

@@ -1,39 +0,0 @@
-# frozen_string_literal: true
-
-$LOAD_PATH.unshift File.expand_path("../../lib", __dir__)
-
-require 'async'
-require 'async/io/stream'
-require 'async/http/endpoint'
-require 'protocol/http1/connection'
-require 'pry'
-
-Async do
-	endpoint = Async::HTTP::Endpoint.parse("https://www.google.com/search?q=kittens", alpn_protocols: ["http/1.1"])
-	
-	peer = endpoint.connect
-	
-	puts "Connected to #{peer} #{peer.remote_address.inspect}"
-	
-	# IO Buffering...
-	stream = Async::IO::Stream.new(peer)
-	client = Protocol::HTTP1::Connection.new(stream)
-	
-	def client.read_line
-		@stream.read_until(Protocol::HTTP1::Connection::CRLF) or raise EOFError
-	end
-	
-	puts "Writing request..."
-	client.write_request("www.google.com", "GET", "/search?q=kittens", "HTTP/1.1", [["Accept", "*/*"]])
-	client.write_body(nil)
-	
-	puts "Reading response..."
-	response = client.read_response("GET")
-	
-	puts "Got response: #{response.inspect}"
-	
-	puts "Closing client..."
-	client.close
-end
-
-puts "Exiting."

data/protocol-http1.gemspec

@@ -1,31 +0,0 @@
-
-require_relative 'lib/protocol/http1/version'
-
-Gem::Specification.new do |spec|
-	spec.name          = "protocol-http1"
-	spec.version       = Protocol::HTTP1::VERSION
-	spec.authors       = ["Samuel Williams"]
-	spec.email         = ["samuel.williams@oriontransfer.co.nz"]
-	
-	spec.summary       = "A low level implementation of the HTTP/1 protocol."
-	spec.homepage      = "https://github.com/socketry/protocol-http1"
-	spec.license       = "MIT"
-	
-	spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
-		`git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
-	end
-	
-	spec.required_ruby_version = "~> 2.4"
-	
-	spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-	spec.require_paths = ["lib"]
-	
-	spec.add_dependency "protocol-http", "~> 0.18"
-	
-	spec.add_development_dependency "covered"
-	spec.add_development_dependency "bundler"
-	spec.add_development_dependency "bake-bundler"
-	spec.add_development_dependency "rspec", "~> 3.0"
-	spec.add_development_dependency "rspec-memory", "~> 1.0"
-	spec.add_development_dependency "rspec-files", "~> 1.0"
-end