diff --git a/CHANGELOG.md b/CHANGELOG.md index 830f087e..e46b399b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,3 +1,7 @@ +# 4.10.1 + +- Fix an object ownership/sharing bug where the Rails log level was erroneously being set to `WARN` when initialising Sentry. + # 4.10.0 - Reduce log level for the Sentry gem from `INFO` to `WARN` to avoid polluting logs with uninformative messages. This only affects log messages from the Sentry gem itself, which go to `stdout`. diff --git a/lib/govuk_app_config/govuk_error.rb b/lib/govuk_app_config/govuk_error.rb index d8adc748..e1d98076 100644 --- a/lib/govuk_app_config/govuk_error.rb +++ b/lib/govuk_app_config/govuk_error.rb @@ -40,8 +40,6 @@ def self.configure Sentry.init do |sentry_config| config = Configuration.new(sentry_config) - # Avoid "Sending envelope with items ... to Sentry" logspew on stdout. - config.logger.level = Sentry::Logger::WARN yield config if block_given? end end diff --git a/lib/govuk_app_config/govuk_error/configuration.rb b/lib/govuk_app_config/govuk_error/configuration.rb index 135ab407..c8abd1f7 100644 --- a/lib/govuk_app_config/govuk_error/configuration.rb +++ b/lib/govuk_app_config/govuk_error/configuration.rb @@ -74,6 +74,10 @@ def set_up_defaults "GdsApi::ContentStore::ItemNotFound", ] + # Avoid "Sending envelope with items ... to Sentry" logspew, since we + # don't use Sentry's automatic session tracking. + self.auto_session_tracking = false + @before_send_callbacks = [ ignore_excluded_exceptions_in_data_sync, increment_govuk_statsd_counters, diff --git a/lib/govuk_app_config/version.rb b/lib/govuk_app_config/version.rb index c3d37d7a..5fcd1025 100644 --- a/lib/govuk_app_config/version.rb +++ b/lib/govuk_app_config/version.rb @@ -1,3 +1,3 @@ module GovukAppConfig - VERSION = "4.10.0".freeze + VERSION = "4.10.1".freeze end