spider-gazelle 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6a7447a407b6297445d56cc9dc47021b598375af
-  data.tar.gz: 16285931940c9e329b8f24d58659eca23e329822
+  metadata.gz: 06ce9d0c2e12d8dda15fd597dbba9026d8848520
+  data.tar.gz: 3c649fbf8734a3f73e50073d3cf83efdc590e2a6
 SHA512:
-  metadata.gz: 5d4fbb8493cd111f15fbfe6f6d718aa3afac4ddc13f3ce6d952c6b13cb1c0343e700db29197594f8364a3caea933141341584603ffa1bbc91dbf1faee01972d6
-  data.tar.gz: 46dbe2558d2678bf5aba39648cd8ac5fa64aa9c03b151cc2a4083484a1a3891ea5b9127d903e0519c1a6ae8b6b82443d64e1de3b6145915778a68733e794e8a7
+  metadata.gz: 438f525a31e89401f9c86dd57c50a97c3a558eca28e04dc82e1173cd9af801e3de899282f55a387fccbc2ff2d13b0da5ca74645320edc8fc2a0c4fb7b0756d32
+  data.tar.gz: 4a52edec124cb480ab4c51a9c584923529045f51007ee3213ef58a77669896ef51569d461ffd9d8e12e10148106fb80305bd4f8395e65d2a908771cfef5f88dd

data/lib/spider-gazelle/connection.rb

@@ -30,7 +30,6 @@ module SpiderGazelle
 
       # Used to chain promises (ensures requests are processed in order)
       @process_next = method :process_next
-      @write_chunk = method :write_chunk
       # Keep track of work queue head to prevent unintentional GC
       @current_worker = queue
       # Start queue with an existing resolved promise (::Libuv::Q::ResolvedPromise.new(@loop, true))
@@ -166,7 +165,6 @@ module SpiderGazelle
           end
 
           return promise
-          # NOTE:: Somehow getting to here with a nil request... needs investigation
         elsif result
           # clear any cached responses just in case
           # could be set by error in the rack application
@@ -247,6 +245,7 @@ module SpiderGazelle
         write_headers status, headers
 
         # Stream the response
+        @write_chunk ||= method :write_chunk
         body.each &@write_chunk
 
         if @request.deferred.nil?
@@ -269,7 +268,7 @@ module SpiderGazelle
 
         value.split(NEWLINE).each do |unique_value|
           header << key
-          header << COLON
+          header << COLON_SPACE
           header << unique_value
           header << LINE_END
         end
@@ -332,6 +331,7 @@ module SpiderGazelle
         deferred.resolve true
       else
         # Send the chunks provided
+        @write_chunk ||= method :write_chunk
         body.each &@write_chunk
         body.close if body.respond_to?(:close)
       end

data/lib/spider-gazelle/const.rb

@@ -27,7 +27,7 @@ module SpiderGazelle
   # REMOTE_USER, or REMOTE_HOST parameters since those are either a security problem or
   # too taxing on performance.
   module Const
-    SPIDER_GAZELLE_VERSION = VERSION = "1.0.0".freeze
+    SPIDER_GAZELLE_VERSION = VERSION = "1.0.1".freeze
     # CODE_NAME = "Earl of Sandwich Partition"
     SERVER = "SpiderGazelle".freeze
 
@@ -147,6 +147,8 @@ module SpiderGazelle
     # SPIDER_GAZELLE_SOCKET = "spider-gazelle.socket".freeze
     # SPIDER_GAZELLE_CONFIG = "spider-gazelle.config".freeze
 
+    ASCII_8BIT = "ASCII-8BIT".freeze
+
     HTTP = "http".freeze
     HTTPS = "https".freeze
 
@@ -176,9 +178,14 @@ module SpiderGazelle
     CLOSE_CHUNKED = "0\r\n\r\n".freeze
 
     COMMA = ", ".freeze
-    COLON = ": ".freeze
+    COLON_SPACE = ": ".freeze
+    COLON = ":".freeze
+    DASH = "-".freeze
+    UNDERSCORE = "_".freeze
+    SPACE = " ".freeze
     NEWLINE = "\n".freeze
     EMPTY = "".freeze
+    QUESTION_MARK = "?".freeze
 
     ZERO = "0".freeze
 

data/lib/spider-gazelle/gazelle.rb

@@ -81,7 +81,7 @@ module SpiderGazelle
       else
         header = req.header
         header.upcase!
-        header.gsub!('-', '_')
+        header.gsub!(DASH, UNDERSCORE)
         header.prepend(HTTP_META)
         header.freeze
         if req.env[header]
@@ -131,7 +131,7 @@ module SpiderGazelle
       end
 
       # Data == "TLS_indicator Port APP_ID"
-      tls, port, app_id = data.split(' ', 3)
+      tls, port, app_id = data.split(SPACE, 3)
       app = @app_cache[app_id.to_sym] ||= AppStore.get(app_id)
       inst = @parser_cache.pop || ::HttpParser::Parser.new_instance(&@set_instance_type)
 
@@ -139,7 +139,7 @@ module SpiderGazelle
       socket.progress @on_progress
       # TODO:: Allow some globals for supplying the certs
       #  --> We could store these in the AppStore
-      socket.start_tls(:server => true) if tls == 'T'
+      socket.start_tls(:server => true) if tls == USE_TLS
 
       # Keep track of the connection
       connection = Connection.new self, @gazelle, socket, port, inst, app, @connection_queue

data/lib/spider-gazelle/request.rb

@@ -29,7 +29,6 @@ module SpiderGazelle
       @body = ''
       @header = ''
       @url = ''
-      @execute = method :execute
       @env = PROTO_ENV.dup
       @loop = connection.loop
       @env[SERVER_PORT] = connection.port
@@ -44,11 +43,11 @@ module SpiderGazelle
       @env[REQUEST_URI] = @url.freeze
 
       # For Rack::Lint on 1.9, ensure that the encoding is always for spec
-      @body.force_encoding('ASCII-8BIT') if @body.respond_to?(:force_encoding)
+      @body.force_encoding(ASCII_8BIT) if @body.respond_to?(:force_encoding)
       @env[RACK_INPUT] = StringIO.new @body
 
       # Break the request into its components
-      query_start  = @url.index '?'
+      query_start  = @url.index QUESTION_MARK
       if query_start
         path = @url[0...query_start].freeze
         @env[PATH_INFO] = path
@@ -57,12 +56,12 @@ module SpiderGazelle
       else
         @env[PATH_INFO] = @url
         @env[REQUEST_PATH] = @url
-        @env[QUERY_STRING] = ''
+        @env[QUERY_STRING] = EMPTY
       end
 
       # Grab the host name from the request
       if host = @env[HTTP_HOST]
-        if colon = host.index(':')
+        if colon = host.index(COLON)
           @env[SERVER_NAME] = host[0, colon]
           @env[SERVER_PORT] = host[colon+1, host.bytesize]
         else
@@ -81,7 +80,7 @@ module SpiderGazelle
       end
 
       # Execute the request
-      @response = catch :async, &@execute
+      @response = catch(:async) { @app.call @env }
       if @response.nil? || @response[0] == -1
         @deferred = @loop.defer
 
@@ -96,12 +95,6 @@ module SpiderGazelle
 
     protected
 
-    # Execute the request then return the result to the event loop
-    def execute(*args)
-      result = @app.call @env
-      result
-    end
-
     def hijack
       @hijacked = @loop.defer
       @env[HIJACK_IO] = @hijacked.promise

data/lib/spider-gazelle/spider.rb

@@ -221,7 +221,9 @@ module SpiderGazelle
     # Called from the binding for sending to gazelles
     def delegate(client, tls, port, app_id)
       indicator = tls ? USE_TLS : NO_TLS
-      @select_handler.next.write2(client, "#{indicator} #{port} #{app_id}")
+      @select_handler.next.write2(client, "#{indicator} #{port} #{app_id}").finally do
+        client.close
+      end
     end
 
     def direct_delegate(client, tls, port, app_id)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spider-gazelle
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Stephen von Takach
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-20 00:00:00.000000000 Z
+date: 2014-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake