protocol-http1 0.11.0 → 0.13.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dd207477ff1c6e86ccc2a90b3ec7ae27c7a3f5dbfd58e42cf78fb841638d43c1
-  data.tar.gz: 797536146daeb57ce4b40cf4e71bbcea02cc49067ecdd05d1c1b9c55f768212d
+  metadata.gz: 95b00d4f51351bedac2db6576b3117c37f6de76ab3d06b7535c19bdb45f8c3c5
+  data.tar.gz: d715bee3904278d4e043029aac21032316b7e43f57987a2b50ab5d51bf5d2e77
 SHA512:
-  metadata.gz: 0efd8f411bc383b88694116c674376acd8d9fd9871aa4d3a89bfa95a39b0124aa2577c1c3767a11eed91f310bb0581f07c8f80b20ab2a79033db534028b346ce
-  data.tar.gz: 4e1836661c69a4dbe0cc42ac961afd76153e07f5a41f83428a8d37673924a05cbc9795daa1cc14f530bb38815cee374b61e2dbe105e6df21fafade35a103d6e2
+  metadata.gz: 481a4d626cdaa358f53f13c3d000aab1512bde51a6bf9f18d5e72310fd6011796b9c037492ed544bb63ebe26a5b009a922d69531440fef1cd65796a236fef812
+  data.tar.gz: e14d62075c96f88ed45bdaedf63b3740ee55c94478c76bc12368671134b1172a83d85480904bdb109d7f1107b3e18718abe9257565f55570699d9df602e203e8

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

@@ -90,8 +90,6 @@ module Protocol
 				end
 				
 				def read_trailers
-					@headers.trailers!
-					
 					while line = read_line
 						# Empty line indicates end of headers:
 						break if line.empty?

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
@@ -309,21 +312,25 @@ module Protocol
 			
 			def write_chunked_body(body, head, trailers = 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
@@ -342,14 +349,15 @@ 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
 				
@@ -357,12 +365,16 @@ module Protocol
 			end
 			
 			def write_body(version, body, head = false, trailers = nil)
-				if body.nil? or body.empty?
+				if body.nil?
 					write_connection_header(version)
 					write_empty_body(body)
-				elsif length = body.length
+				elsif length = body.length and trailers.nil?
 					write_connection_header(version)
 					write_fixed_length_body(body, length, head)
+				elsif body.empty?
+					# Even thought this code is the same as the first clause `body.nil?`, HEAD responses have an empty body but still carry a content length. `write_fixed_length_body` takes care of this appropriately.
+					write_connection_header(version)
+					write_empty_body(body)
 				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.

data/lib/protocol/http1/version.rb

@@ -22,6 +22,6 @@
 
 module Protocol
 	module HTTP1
-		VERSION = "0.11.0"
+		VERSION = "0.13.2"
 	end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: protocol-http1
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.13.2
 platform: ruby
 authors:
 - Samuel Williams
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-06 00:00:00.000000000 Z
+date: 2020-11-25 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.16.3
+        version: '0.19'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.16.3
-- !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.19'
 - !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
@@ -151,7 +128,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+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.16.3"
-	
-	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