build_status_server 0.12 → 0.13

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    build_status_server (0.12)
+    build_status_server (0.13)
       json
 
 GEM

data/lib/build_status_server/server.rb

@@ -67,7 +67,12 @@ The address configured is not available (#{address})
 
 
     def process_job(data = "{}")
-      job = JSON.parse(data)
+      job = begin
+              JSON.parse(data)
+            rescue JSON::ParserError => ex
+              STDERR.puts "Invalid JSON! (Or at least JSON wasn't able to parse it...)\nReceived: #{data}"
+              return false
+            end
 
       build_name = job["name"]
 

data/lib/build_status_server/version.rb

@@ -1,3 +1,3 @@
 module BuildStatusServer
-  VERSION = "0.12"
+  VERSION = "0.13"
 end

data/spec/lib/build_status_server/server_spec.rb

@@ -224,6 +224,11 @@ describe BuildStatusServer::Server do
         server.send(:process_job).should be_false
       end
 
+      it "doesn't die if invalid JSON is passed in" do
+        STDERR.should_receive(:puts).with("Invalid JSON! (Or at least JSON wasn't able to parse it...)\nReceived: {b => \"123\"}")
+        expect { server.send(:process_job, '{b => "123"}') }.should_not raise_error(JSON::ParserError)
+      end
+
       it "returns false and says so on stderr if payload doesn't have a hash for build" do
         server.should_receive(:should_process_build).and_return(true)
         JSON.should_receive(:parse).and_return({"build" => "Not a hash!"})

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: build_status_server
 version: !ruby/object:Gem::Version
-  version: '0.12'
+  version: '0.13'
   prerelease: 
 platform: ruby
 authors: