pact_broker 2.75.0 → 2.78.0

data/lib/pact_broker/api/resources/default_base_resource.rb

@@ -68,12 +68,12 @@ module PactBroker
         alias_method :path_info, :identifier_from_path
 
         def base_url
-          request.env["pactbroker.base_url"] || PactBroker.configuration.base_url || request.base_uri.to_s.chomp('/')
+          request.env["pactbroker.base_url"] || request.base_uri.to_s.chomp('/')
         end
 
         # See comments for base_url in lib/pact_broker/doc/controllers/app.rb
         def ui_base_url
-          request.env["pactbroker.base_url"] || PactBroker.configuration.base_url || ''
+          request.env["pactbroker.base_url"] || ''
         end
 
         def charsets_provided
@@ -149,6 +149,10 @@ module PactBroker
           identifier_from_path[:consumer_version_number]
         end
 
+        def pacticipant_version_number
+          identifier_from_path[:pacticipant_version_number]
+        end
+
         def consumer_specified?
           identifier_from_path.key?(:consumer_name)
         end
@@ -239,6 +243,15 @@ module PactBroker
         def decorator_class(name)
           application_context.decorator_configuration.class_for(name)
         end
+
+        def validation_errors_for_schema?(schema, params_to_validate = params)
+          if (errors = schema.call(params_to_validate)).any?
+            set_json_validation_error_messages(errors)
+            true
+          else
+            false
+          end
+        end
       end
     end
   end

data/lib/pact_broker/api/resources/environment.rb

@@ -0,0 +1,76 @@
+require 'pact_broker/api/resources/base_resource'
+require 'pact_broker/api/resources/environment'
+
+module PactBroker
+  module Api
+    module Resources
+      class Environment < BaseResource
+        def content_types_provided
+          [["application/hal+json", :to_json]]
+        end
+
+        def content_types_accepted
+          [["application/json", :from_json]]
+        end
+
+        def allowed_methods
+          ["GET", "PUT", "DELETE", "OPTIONS"]
+        end
+
+        def resource_exists?
+          !!environment
+        end
+
+        def malformed_request?
+          if request.put? && environment
+            invalid_json? || validation_errors_for_schema?(schema, params.merge(uuid: uuid))
+          else
+            false
+          end
+        end
+
+        def from_json
+          if environment
+            @environment = update_environment
+            response.body = to_json
+          else
+            response.code = 404
+          end
+        end
+
+        def policy_name
+          :'deployments::environment'
+        end
+
+        def to_json
+          decorator_class(:environment_decorator).new(environment).to_json(decorator_options)
+        end
+
+        def parsed_environment
+          @parsed_environment ||= decorator_class(:environment_decorator).new(PactBroker::Deployments::Environment.new).from_json(request_body)
+        end
+
+        def environment
+          @environment ||= environment_service.find(uuid)
+        end
+
+        def delete_resource
+          environment_service.delete(uuid)
+          true
+        end
+
+        def uuid
+          identifier_from_path[:environment_uuid]
+        end
+
+        def update_environment
+          environment_service.update(uuid, parsed_environment)
+        end
+
+        def schema
+          PactBroker::Api::Contracts::EnvironmentSchema
+        end
+      end
+    end
+  end
+end

data/lib/pact_broker/api/resources/environments.rb

@@ -0,0 +1,75 @@
+require 'pact_broker/api/resources/base_resource'
+require 'pact_broker/api/resources/environment'
+require 'pact_broker/api/contracts/environment_schema'
+
+module PactBroker
+  module Api
+    module Resources
+      class Environments < BaseResource
+        def content_types_provided
+          [["application/hal+json", :to_json]]
+        end
+
+        def content_types_accepted
+          [["application/json", :from_json]]
+        end
+
+        def allowed_methods
+          ["GET", "POST", "OPTIONS"]
+        end
+
+        def resource_exists?
+          true
+        end
+
+        def post_is_create?
+          true
+        end
+
+        def malformed_request?
+          if request.post?
+            invalid_json? || validation_errors_for_schema?(schema, params.merge(uuid: uuid))
+          else
+            false
+          end
+        end
+
+        def create_path
+          environment_url(OpenStruct.new(uuid: uuid), base_url)
+        end
+
+        def from_json
+          response.body = decorator_class(:environment_decorator).new(create_environment).to_json(decorator_options)
+        end
+
+        def policy_name
+          :'deployments::environment'
+        end
+
+        def to_json
+          decorator_class(:environments_decorator).new(environments).to_json(decorator_options)
+        end
+
+        def parsed_environment
+          @parsed_environment ||= decorator_class(:environment_decorator).new(PactBroker::Deployments::Environment.new).from_json(request_body)
+        end
+
+        def create_environment
+          environment_service.create(uuid, parsed_environment)
+        end
+
+        def uuid
+          @uuid ||= environment_service.next_uuid
+        end
+
+        def environments
+          @environments ||= environment_service.find_all
+        end
+
+        def schema
+          PactBroker::Api::Contracts::EnvironmentSchema
+        end
+      end
+    end
+  end
+end

data/lib/pact_broker/api/resources/index.rb

@@ -110,6 +110,12 @@ module PactBroker
               title: "Get, create or delete a tag for a pacticipant version",
               templated: true
             },
+            'pb:pacticipant-version' =>
+            {
+              href: base_url + '/pacticipants/{pacticipant}/versions/{version}',
+              title: "Get, create or delete a pacticipant version",
+              templated: true
+            },
             'pb:metrics' =>
             {
               href: base_url + '/metrics',
@@ -144,6 +150,20 @@ module PactBroker
             }]
           }
 
+          if PactBroker.feature_enabled?(:environments)
+            links_hash['pb:environments'] = {
+              title: "Environments",
+              href: environments_url(base_url),
+              templated: false
+            }
+
+            links_hash['pb:environment'] = {
+              title: "Environment",
+              href: environments_url(base_url) + "/{uuid}",
+              templated: true
+            }
+          end
+
           if PactBroker.feature_enabled?('disable_pacts_for_verification', true)
             links_hash.delete('pb:provider-pacts-for-verification')
             links_hash.delete('beta:provider-pacts-for-verification')

data/lib/pact_broker/api/resources/latest_version.rb

@@ -0,0 +1,27 @@
+require 'pact_broker/api/resources/version'
+
+module PactBroker
+  module Api
+    module Resources
+      class LatestVersion < Version
+        def content_types_accepted
+          []
+        end
+
+        def allowed_methods
+          ["GET", "OPTIONS"]
+        end
+
+        private
+
+        def version
+          if identifier_from_path[:tag]
+            @version ||= version_service.find_by_pacticipant_name_and_latest_tag(identifier_from_path[:pacticipant_name], identifier_from_path[:tag])
+          else
+            @version ||= version_service.find_latest_by_pacticpant_name(identifier_from_path)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/pact_broker/api/resources/provider_pacts_for_verification.rb

@@ -38,6 +38,7 @@ module PactBroker
         def pacts
           @pacts ||= pact_service.find_for_verification(
             provider_name,
+            parsed_query_params.provider_version_branch,
             parsed_query_params.provider_version_tags,
             parsed_query_params.consumer_version_selectors,
             {

data/lib/pact_broker/api/resources/verifications.rb

@@ -50,7 +50,7 @@ module PactBroker
         end
 
         def from_json
-          verification = verification_service.create(next_verification_number, verification_params, pact, webhook_options)
+          verification = verification_service.create(next_verification_number, verification_params, pact, event_context, webhook_options)
           response.body = decorator_for(verification).to_json(decorator_options)
           true
         end
@@ -81,11 +81,14 @@ module PactBroker
           metadata[:wip] == 'true'
         end
 
+        def event_context
+          metadata
+        end
+
         def webhook_options
           {
             database_connector: database_connector,
             webhook_execution_configuration: webhook_execution_configuration
-                                      .with_webhook_context(metadata)
           }
         end
 

data/lib/pact_broker/api/resources/version.rb

@@ -10,20 +10,32 @@ module PactBroker
           [["application/hal+json", :to_json]]
         end
 
+        def content_types_accepted
+          [["application/json", :from_json]]
+        end
+
         def allowed_methods
-          ["GET", "DELETE", "OPTIONS"]
+          ["GET", "PUT", "DELETE", "OPTIONS"]
         end
 
         def resource_exists?
           !!version
         end
 
+        def from_json
+          response_code = version ? 200 : 201
+          parsed_version = Decorators::VersionDecorator.new(OpenStruct.new).from_json(request_body)
+          @version = version_service.create_or_overwrite(pacticipant_name, pacticipant_version_number, parsed_version)
+          response.body = to_json
+          response_code
+        end
+
         def to_json
           decorator_class(:version_decorator).new(version).to_json(decorator_options)
         end
 
         def delete_resource
-          version_service.delete version
+          version_service.delete(version)
           true
         end
 
@@ -34,13 +46,7 @@ module PactBroker
         private
 
         def version
-          if identifier_from_path[:tag]
-            @version ||= version_service.find_by_pacticipant_name_and_latest_tag(identifier_from_path[:pacticipant_name], identifier_from_path[:tag])
-          elsif identifier_from_path[:pacticipant_version_number]
-            @version ||= version_service.find_by_pacticipant_name_and_number(identifier_from_path)
-          else
-            @version ||= version_service.find_latest_by_pacticpant_name(identifier_from_path)
-          end
+          @version ||= version_service.find_by_pacticipant_name_and_number(identifier_from_path)
         end
       end
     end

data/lib/pact_broker/api/resources/webhook_execution.rb

@@ -26,7 +26,7 @@ module PactBroker
         end
 
         def process_post
-          webhook_execution_result = webhook_service.test_execution(webhook, webhook_execution_configuration)
+          webhook_execution_result = webhook_service.test_execution(webhook, webhook_execution_configuration.webhook_context, webhook_execution_configuration)
           response.headers['Content-Type'] = 'application/hal+json;charset=utf-8'
           response.body = post_response_body(webhook_execution_result)
           true

data/lib/pact_broker/app.rb

@@ -5,6 +5,7 @@ require 'pact_broker/logging/default_formatter'
 require 'pact_broker/policies'
 require 'rack-protection'
 require 'rack/hal_browser'
+require 'rack/pact_broker/set_base_url'
 require 'rack/pact_broker/add_pact_broker_version_header'
 require 'rack/pact_broker/convert_file_extension_to_accept_header'
 require 'rack/pact_broker/database_transaction'
@@ -189,6 +190,8 @@ module PactBroker
       @app_builder.use Rack::Static, :urls => ["/stylesheets", "/css", "/fonts", "/js", "/javascripts", "/images"], :root => PactBroker.project_root.join("public")
       @app_builder.use Rack::Static, :urls => ["/favicon.ico"], :root => PactBroker.project_root.join("public/images"), header_rules: [[:all, {'Content-Type' => 'image/x-icon'}]]
       @app_builder.use Rack::PactBroker::ConvertFileExtensionToAcceptHeader
+      # Rack::PactBroker::SetBaseUrl needs to be before the Rack::PactBroker::HalBrowserRedirect
+      @app_builder.use Rack::PactBroker::SetBaseUrl, configuration.base_url
 
       if configuration.use_hal_browser
         logger.info "Mounting HAL browser"

data/lib/pact_broker/certificates/certificate.rb

@@ -9,7 +9,7 @@ end
 
 # Table: certificates
 # Columns:
-#  id          | integer                     | PRIMARY KEY DEFAULT nextval('certificates_id_seq'::regclass)
+#  id          | integer                     | PRIMARY KEY GENERATED BY DEFAULT AS IDENTITY
 #  uuid        | text                        | NOT NULL
 #  description | text                        |
 #  content     | text                        | NOT NULL

data/lib/pact_broker/config/setting.rb

@@ -73,7 +73,7 @@ end
 
 # Table: config
 # Columns:
-#  id         | integer                     | PRIMARY KEY DEFAULT nextval('config_id_seq'::regclass)
+#  id         | integer                     | PRIMARY KEY GENERATED BY DEFAULT AS IDENTITY
 #  name       | text                        | NOT NULL
 #  type       | text                        | NOT NULL
 #  value      | text                        |

data/lib/pact_broker/config/space_delimited_integer_list.rb

@@ -0,0 +1,25 @@
+module PactBroker
+  module Config
+    class SpaceDelimitedIntegerList < Array
+      def initialize list
+        super(list)
+      end
+
+      def self.integer?(string)
+        (Integer(string) rescue nil) != nil
+      end
+
+      def self.parse(string)
+        array = (string || '')
+                    .split(' ')
+                    .select { |word| integer?(word) }
+                    .collect(&:to_i)
+        SpaceDelimitedIntegerList.new(array)
+      end
+
+      def to_s
+        collect(&:to_s).join(' ')
+      end
+    end
+  end
+end

data/lib/pact_broker/configuration.rb

@@ -1,6 +1,7 @@
 require 'pact_broker/version'
 require 'pact_broker/error'
 require 'pact_broker/config/space_delimited_string_list'
+require 'pact_broker/config/space_delimited_integer_list'
 require 'semantic_logger'
 
 module PactBroker
@@ -30,6 +31,7 @@ module PactBroker
       :webhook_http_method_whitelist,
       :webhook_scheme_whitelist,
       :webhook_host_whitelist,
+      :webhook_http_code_success,
       :base_equality_only_on_content_that_affects_verification_results,
       :seed_example_data,
       :badge_provider_mode,
@@ -49,7 +51,7 @@ module PactBroker
     attr_accessor :check_for_potential_duplicate_pacticipant_names
     attr_accessor :webhook_retry_schedule
     attr_accessor :user_agent
-    attr_reader :webhook_http_method_whitelist, :webhook_scheme_whitelist, :webhook_host_whitelist
+    attr_reader :webhook_http_method_whitelist, :webhook_scheme_whitelist, :webhook_host_whitelist, :webhook_http_code_success
     attr_accessor :semver_formats
     attr_accessor :enable_public_badge_access, :shields_io_base_url, :badge_provider_mode
     attr_accessor :disable_ssl_verification
@@ -105,6 +107,7 @@ module PactBroker
       config.check_for_potential_duplicate_pacticipant_names = true
       config.disable_ssl_verification = false
       config.webhook_http_method_whitelist = ['POST']
+      config.webhook_http_code_success = [200, 201, 202, 203, 204, 205, 206]
       config.webhook_scheme_whitelist = ['https']
       config.webhook_host_whitelist = []
       # TODO get rid of unsafe-inline
@@ -240,6 +243,10 @@ module PactBroker
       @webhook_http_method_whitelist = parse_space_delimited_string_list_property('webhook_http_method_whitelist', webhook_http_method_whitelist)
     end
 
+    def webhook_http_code_success= webhook_http_code_success
+      @webhook_http_code_success = parse_space_delimited_integer_list_property('webhook_http_code_success', webhook_http_code_success)
+    end
+
     def webhook_scheme_whitelist= webhook_scheme_whitelist
       @webhook_scheme_whitelist = parse_space_delimited_string_list_property('webhook_scheme_whitelist', webhook_scheme_whitelist)
     end
@@ -269,5 +276,15 @@ module PactBroker
         raise ConfigurationError.new("Pact Broker configuration property `#{property_name}` must be a space delimited String or an Array")
       end
     end
+
+    def parse_space_delimited_integer_list_property property_name, property_value
+      case property_value
+      when String then Config::SpaceDelimitedIntegerList.parse(property_value)
+      when Array then Config::SpaceDelimitedIntegerList.new(property_value)
+      else
+        raise ConfigurationError.new("Pact Broker configuration property `#{property_name}` must be a space delimited String or an Array with Integer values")
+      end
+    end
+
   end
 end