big_brother 0.2.1 → 0.3.0

data/lib/big_brother/health_fetcher.rb

@@ -1,12 +1,15 @@
 module BigBrother
   class HealthFetcher
     def self.current_health(address, port, path)
-      http_response = EventMachine::HttpRequest.new("http://#{address}:#{port}#{path}").get
+      response = EventMachine::HttpRequest.new("http://#{address}:#{port}#{path}").get
+      response.response_header.status == 200 ? _parse_health(response) : 0
+    end
 
-      if http_response.response_header.has_key?('X_HEALTH')
-        http_response.response_header['X_HEALTH'].to_i
+    def self._parse_health(response)
+      if response.response_header.has_key?('X_HEALTH')
+        response.response_header['X_HEALTH'].to_i
       else
-        http_response.response.slice(/Health: (\d+)/, 1).to_i
+        response.response.slice(/Health: (\d+)/, 1).to_i
       end
     end
   end

data/lib/big_brother/version.rb

@@ -1,3 +1,3 @@
 module BigBrother
-  VERSION = "0.2.1"
+  VERSION = "0.3.0"
 end

data/spec/big_brother/health_fetcher_spec.rb

@@ -5,7 +5,7 @@ describe BigBrother::HealthFetcher do
     run_in_reactor
 
     it "returns its health" do
-      StubServer.new(<<-HTTP, 0.25)
+      StubServer.new(<<-HTTP)
 HTTP/1.0 200 OK
 Connection: close
 
@@ -14,8 +14,18 @@ HTTP
       BigBrother::HealthFetcher.current_health("127.0.0.1", 8081, "/test/status").should == 59
     end
 
+    it "returns 0 when the HTTP status code is not 200" do
+      StubServer.new(<<-HTTP)
+HTTP/1.0 503 OK
+Connection: close
+
+Health: 19
+HTTP
+      BigBrother::HealthFetcher.current_health("127.0.0.1", 8081, "/test/status").should == 0
+    end
+
     it "returns 0 for an unknown service" do
-      StubServer.new(<<-HTTP, 0.25)
+      StubServer.new(<<-HTTP)
 HTTP/1.0 503 Service Unavailable
 Connection: close
 HTTP
@@ -27,7 +37,7 @@ HTTP
     end
 
     it "returns the health if it is passed in a header" do
-      StubServer.new(<<-HTTP, 0.25)
+      StubServer.new(<<-HTTP)
 HTTP/1.0 200 OK
 Connection: close
 X-Health: 61

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: big_brother
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-05-16 00:00:00.000000000 Z
+date: 2012-05-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thin
-  requirement: &70175345459640 !ruby/object:Gem::Requirement
+  requirement: &70147105239120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 1.3.1
   type: :runtime
   prerelease: false
-  version_requirements: *70175345459640
+  version_requirements: *70147105239120
 - !ruby/object:Gem::Dependency
   name: async-rack
-  requirement: &70175345458880 !ruby/object:Gem::Requirement
+  requirement: &70147105238560 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 0.5.1
   type: :runtime
   prerelease: false
-  version_requirements: *70175345458880
+  version_requirements: *70147105238560
 - !ruby/object:Gem::Dependency
   name: sinatra
-  requirement: &70175345458360 !ruby/object:Gem::Requirement
+  requirement: &70147105237960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: '1.0'
   type: :runtime
   prerelease: false
-  version_requirements: *70175345458360
+  version_requirements: *70147105237960
 - !ruby/object:Gem::Dependency
   name: rack-fiber_pool
-  requirement: &70175345457700 !ruby/object:Gem::Requirement
+  requirement: &70147105237200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +54,10 @@ dependencies:
         version: '0.9'
   type: :runtime
   prerelease: false
-  version_requirements: *70175345457700
+  version_requirements: *70147105237200
 - !ruby/object:Gem::Dependency
   name: eventmachine
-  requirement: &70175345456980 !ruby/object:Gem::Requirement
+  requirement: &70147105236460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>'
@@ -68,10 +68,10 @@ dependencies:
         version: 1.0.0.beta.100
   type: :runtime
   prerelease: false
-  version_requirements: *70175345456980
+  version_requirements: *70147105236460
 - !ruby/object:Gem::Dependency
   name: em-http-request
-  requirement: &70175345455800 !ruby/object:Gem::Requirement
+  requirement: &70147105235180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -79,10 +79,10 @@ dependencies:
         version: '1.0'
   type: :runtime
   prerelease: false
-  version_requirements: *70175345455800
+  version_requirements: *70147105235180
 - !ruby/object:Gem::Dependency
   name: em-synchrony
-  requirement: &70175345454760 !ruby/object:Gem::Requirement
+  requirement: &70147105234240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -90,10 +90,10 @@ dependencies:
         version: '1.0'
   type: :runtime
   prerelease: false
-  version_requirements: *70175345454760
+  version_requirements: *70147105234240
 - !ruby/object:Gem::Dependency
   name: em-resolv-replace
-  requirement: &70175345453880 !ruby/object:Gem::Requirement
+  requirement: &70147105233120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -101,10 +101,10 @@ dependencies:
         version: '1.1'
   type: :runtime
   prerelease: false
-  version_requirements: *70175345453880
+  version_requirements: *70147105233120
 - !ruby/object:Gem::Dependency
   name: em-syslog
-  requirement: &70175345452920 !ruby/object:Gem::Requirement
+  requirement: &70147105232280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -112,10 +112,10 @@ dependencies:
         version: 0.0.2
   type: :runtime
   prerelease: false
-  version_requirements: *70175345452920
+  version_requirements: *70147105232280
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70175345452120 !ruby/object:Gem::Requirement
+  requirement: &70147105231800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -123,10 +123,10 @@ dependencies:
         version: 2.9.0
   type: :development
   prerelease: false
-  version_requirements: *70175345452120
+  version_requirements: *70147105231800
 - !ruby/object:Gem::Dependency
   name: rack-test
-  requirement: &70175345451620 !ruby/object:Gem::Requirement
+  requirement: &70147105231220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -134,10 +134,10 @@ dependencies:
         version: 0.6.1
   type: :development
   prerelease: false
-  version_requirements: *70175345451620
+  version_requirements: *70147105231220
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70175345451080 !ruby/object:Gem::Requirement
+  requirement: &70147105230600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -145,10 +145,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70175345451080
+  version_requirements: *70147105230600
 - !ruby/object:Gem::Dependency
   name: rake_commit
-  requirement: &70175345450240 !ruby/object:Gem::Requirement
+  requirement: &70147105229780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -156,10 +156,10 @@ dependencies:
         version: '0.13'
   type: :development
   prerelease: false
-  version_requirements: *70175345450240
+  version_requirements: *70147105229780
 - !ruby/object:Gem::Dependency
   name: vagrant
-  requirement: &70175345449640 !ruby/object:Gem::Requirement
+  requirement: &70147105229320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -167,7 +167,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70175345449640
+  version_requirements: *70147105229320
 description: IPVS backend supervisor
 email:
 - code@getbraintree.com