5883ce95ef
The initializers including this were doing so at the top level, so every object loaded after them had a `current_application_settings` method. However, if someone had rack-attack enabled (which was loaded before these initializers), it would try to load the API, and fail, because `Gitlab::CurrentSettings` didn't have that method. To fix this: 1. Don't include `Gitlab::CurrentSettings` at the top level. We do not need `Object.new.current_application_settings` to work. 2. Make `Gitlab::CurrentSettings` explicitly `extend self`, as we already use it like that in several places. 3. Change the initializers to use that new form. |
||
|---|---|---|
| .. | ||
| subscribers | ||
| base_sampler.rb | ||
| delta.rb | ||
| influx_db.rb | ||
| influx_sampler.rb | ||
| instrumentation.rb | ||
| method_call.rb | ||
| metric.rb | ||
| null_metric.rb | ||
| prometheus.rb | ||
| rack_middleware.rb | ||
| requests_rack_middleware.rb | ||
| sidekiq_metrics_exporter.rb | ||
| sidekiq_middleware.rb | ||
| system.rb | ||
| transaction.rb | ||
| unicorn_sampler.rb | ||