fix: gracefully handle scenario where URL supplied in JSON body is not a String

Author: bethesque

lib/pact_broker/api/contracts/request_validations.rb

@@ -25,7 +25,7 @@ def url_valid?
 
         def uri
           URI(url)
-        rescue URI::InvalidURIError
+        rescue URI::InvalidURIError, ArgumentError
           nil
         end
       end

lib/pact_broker/api/contracts/verification_contract.rb

@@ -21,7 +21,7 @@ def not_blank? value
 
             def valid_url? url
               URI(url)
-            rescue URI::InvalidURIError
+            rescue URI::InvalidURIError, ArgumentError
               nil
             end
 

lib/pact_broker/api/contracts/webhook_contract.rb

@@ -109,8 +109,8 @@ def valid_method?(http_method)
               def valid_url?(url)
                 uri = parse_uri(url)
                 uri.scheme && uri.host
-              rescue URI::InvalidURIError
-                false
+              rescue URI::InvalidURIError, ArgumentError
+                nil
               end
 
               def allowed_webhook_method?(http_method)

lib/rack/pact_broker/invalid_uri_protection.rb

@@ -23,7 +23,7 @@ def valid_uri? env
         begin
           parse(::Rack::Request.new(env).url)
           true
-        rescue URI::InvalidURIError
+        rescue URI::InvalidURIError, ArgumentError
           false
         end
       end

spec/lib/pact_broker/api/contracts/verification_contract_spec.rb

@@ -64,6 +64,14 @@ def modify hash, options
             its(:errors) { is_expected.to be_empty }
           end
 
+          context "when the buildURL is not stringable" do
+            let(:build_url) { {} }
+
+            it "has an error" do
+              expect(subject.errors[:build_url]).to include(match("URL"))
+            end
+          end
+
           context "when the providerApplicationVersion is not present" do
             let(:params) { modify valid_params, without: :providerApplicationVersion }
             it "has an error" do