sitehub 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8eab93145788748ecc28ef5f14aea92b7595f846
-  data.tar.gz: 69f240e1d1d707424bf2e6a07e4791b767441427
+  metadata.gz: af77a45a381dc3d5db1d15d6d4934dd97fcfcf14
+  data.tar.gz: 365f724e5ecc280935665cadc36dbab16b45fe34
 SHA512:
-  metadata.gz: a4d317b035c62e2513261872065ef428ae0342fc8a40c5807b8b597908d53b9f288b6aa310cda0ece34b57e6d1d77068040af6ce27c96fc5e22aaf762e0a1f24
-  data.tar.gz: 4fbf75725640bbc3dcf06e742bb7b549c444048fb8e4f00d0a64888198d62acd1d0b541a4f225b91928107da2e2bc8b70104144ff8697c8035d03c01aef1a266
+  metadata.gz: 5a263ad7fde6a5f68b9c3480ae0765b302918cdbd1eccd30bff05704ed9054c171b7c291932bbc5632b32b30162237c5c586e84e9616392a4604e35cff944e74
+  data.tar.gz: 48358f5f31aae39ff3d70a3bc83e2760b53b37b46046c2cd28eddf53537aa3b607bcc22dce3143a7ac41acc387898660b55d87e51c99196532be4a38412df0f6

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    sitehub (0.4.1)
+    sitehub (0.4.2)
       em-http-request
       em-synchrony
       faraday

data/lib/sitehub/logging/access_logger.rb

@@ -52,7 +52,7 @@ class SiteHub
             env[REQUEST_METHOD],
             env[PATH_INFO],
             env[QUERY_STRING].empty? ? EMPTY_STRING : QUESTION_MARK+env[QUERY_STRING],
-            mapped_request.mapped_url.to_s,
+            mapped_request ? mapped_request.mapped_url.to_s : EMPTY_STRING,
             env[HTTP_VERSION],
             status.to_s[STATUS_RANGE],
             length,

data/lib/sitehub/version.rb

@@ -1,3 +1,3 @@
 class SiteHub
-  VERSION = "0.4.1"
+  VERSION = "0.4.2"
 end

data/spec/sitehub/logging/access_logger_spec.rb

@@ -33,22 +33,14 @@ class SiteHub
         end
       end
 
-      #describe '#before_call' do
-      #  it 'saves the time' do
-      #    now = Time.now
-      #    allow(Time).to receive(:now).and_return(now)
-      #    subject.before_call(:env)
-      #    expect(subject.start_time).to eq(now)
-      #  end
-      #end
-
       describe '#call' do
         let(:env) do
           {
               REQUEST_MAPPING => request_mapping,
               described_class::QUERY_STRING => '',
               described_class::PATH_INFO => 'path_info',
-              Constants::RackHttpHeaderKeys::TRANSACTION_ID => :transaction_id
+              Constants::RackHttpHeaderKeys::TRANSACTION_ID => :transaction_id,
+              described_class::HTTP_VERSION => '1.1'
           }
 
         end
@@ -100,6 +92,14 @@ class SiteHub
           expect(logger.string).to include("#{env[described_class::PATH_INFO]} => #{:mapped_url}")
         end
 
+        context '404 returned, i.e. no downstream call made' do
+          let(:request_mapping) { nil }
+          it 'does not log the down stream url' do
+            subject.call(env)
+            expect(logger.string).to include("=> #{EMPTY_STRING} #{env[described_class::HTTP_VERSION]}")
+          end
+        end
+
       end
       describe '#extract_content_length' do
         context 'content length header not present' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sitehub
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Ladtech
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-08 00:00:00.000000000 Z
+date: 2016-01-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack