Skip to content

Commit 838dec7

Browse files
committed
Change sentry filter parameters (64717)
1 parent b97cb38 commit 838dec7

File tree

2 files changed

+10
-5
lines changed

2 files changed

+10
-5
lines changed

config/environments/test.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
config.action_controller.perform_caching = false
4646

4747
# Raise exceptions instead of rendering exception templates.
48-
config.action_dispatch.show_exceptions = false
48+
config.action_dispatch.show_exceptions = :none
4949

5050
# Disable request forgery protection in test environment.
5151
config.action_controller.allow_forgery_protection = false

config/initializers/sentry.rb

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,16 @@
2929

3030
config.release = Settings.puzzletime.run.full_versioon
3131

32+
filter = ActiveSupport::ParameterFilter.new(Rails.application.config.filter_parameters)
33+
3234
config.before_send = lambda do |event, _hint|
33-
# filter out parameters filtered by Rails
34-
Rails.application.config.filter_parameters.map(&:to_s).each do |param|
35-
event.extra[param] = '[Filtered]' if event&.extra&.key?(param)
36-
end
35+
event.extra = filter.filter(event.extra) if event.extra
36+
event.user = filter.filter(event.user) if event.user
37+
event.contexts = filter.filter(event.contexts) if event.contexts
38+
39+
# Add transaction name because it's not displayed otherwise in glitchtip
40+
event.tags[:transaction] ||= event.transaction if event.transaction
41+
3742
event
3843
end
3944
end

0 commit comments

Comments
 (0)