Use Gitlab::AppLogger in settings

4c524510 · Huzaifa Iftikhar · Dmytro Zaporozhets (DZ) · 4520cd98 · 4c524510 · 4c524510
Commit 4c524510 authored Apr 12, 2021 by Huzaifa Iftikhar Committed by Dmytro Zaporozhets (DZ) Apr 12, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 1 deletion

changelogs/unreleased/use-gitlab-applogger.yml changelogs/unreleased/use-gitlab-applogger.yml +5 -0

config/initializers/1_settings.rb config/initializers/1_settings.rb +1 -1

No files found.
--- a/changelogs/unreleased/use-gitlab-applogger.yml
+++ b/changelogs/unreleased/use-gitlab-applogger.yml
+---
+title: Use Gitlab::AppLogger in settings
+merge_request: 58134
+author: Huzaifa Iftikhar @huzaifaiftikhar
+type: other
--- a/config/initializers/1_settings.rb
+++ b/config/initializers/1_settings.rb
@@ -68,7 +68,7 @@ if Settings.ldap['enabled'] || Rails.env.test?
    server['tls_options'] ||= {}

    if server['ssl_version'] || server['ca_file']
-      Rails.logger.warn 'DEPRECATED: LDAP options `ssl_version` and `ca_file` should be nested within `tls_options`' # rubocop:disable Gitlab/RailsLogger
+      Gitlab::AppLogger.warn 'DEPRECATED: LDAP options `ssl_version` and `ca_file` should be nested within `tls_options`'
    end

    if server['ssl_version']