Skip to content

Include the ee/ directory in backtraces

Sean McGivern requested to merge allow-ee-in-backtraces into master

Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/5204. I could make this just in EE, but it seems fine if CE does this too - it won't do any harm here. No strong preference 🙂

On EE, before:

$ bundle exec rails r 'begin; License.plans_with_feature(:geo); rescue => e; pp Rails.backtrace_cleaner.clean(e.backtrace); end'
[]

After:

$ bundle exec rails r 'begin; License.plans_with_feature(:geo); rescue => e; pp Rails.backtrace_cleaner.clean(e.backtrace); end'
["ee/app/models/license.rb:164:in `plans_with_feature'"]

Merge request reports