Skip to content

Commit

Permalink
feat: allow rack protection to be turned off so that the headers can …
Browse files Browse the repository at this point in the history
…be managed in nginx
  • Loading branch information
bethesque committed Mar 28, 2019
1 parent 0c8106b commit b43e60e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
4 changes: 3 additions & 1 deletion lib/pact_broker/app.rb
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,9 @@ def prepare_app

def configure_middleware
# NOTE THAT NONE OF THIS IS PROTECTED BY AUTH - is that ok?
@app_builder.use Rack::Protection, except: [:path_traversal, :remote_token, :session_hijacking, :http_origin]
if configuration.use_rack_protection
@app_builder.use Rack::Protection, except: [:path_traversal, :remote_token, :session_hijacking, :http_origin]
end
@app_builder.use Rack::PactBroker::InvalidUriProtection
@app_builder.use Rack::PactBroker::StoreBaseURL
@app_builder.use Rack::PactBroker::AddPactBrokerVersionHeader
Expand Down
3 changes: 2 additions & 1 deletion lib/pact_broker/configuration.rb
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class Configuration
:base_equality_only_on_content_that_affects_verification_results
]

attr_accessor :log_dir, :database_connection, :auto_migrate_db, :auto_migrate_db_data, :use_hal_browser, :html_pact_renderer
attr_accessor :log_dir, :database_connection, :auto_migrate_db, :auto_migrate_db_data, :use_hal_browser, :html_pact_renderer, :use_rack_protection
attr_accessor :validate_database_connection_config, :enable_diagnostic_endpoints, :version_parser, :sha_generator
attr_accessor :use_case_sensitive_resource_names, :order_versions_by_date
attr_accessor :check_for_potential_duplicate_pacticipant_names
Expand Down Expand Up @@ -62,6 +62,7 @@ def self.default_configuration
config.log_dir = File.expand_path("./log")
config.auto_migrate_db = true
config.auto_migrate_db_data = true
config.use_rack_protection = true
config.use_hal_browser = true
config.validate_database_connection_config = true
config.enable_diagnostic_endpoints = true
Expand Down

0 comments on commit b43e60e

Please sign in to comment.