Commit 2d7ba6f0 authored by Peter Leitzen's avatar Peter Leitzen

Merge branch 'rails-logger-cop-9' into 'master'

Use applogger in usage_ping and webhook service

See merge request gitlab-org/gitlab!32192
parents 6b180a6c 8651023b
......@@ -28,7 +28,7 @@ class SubmitUsagePingService
true
rescue Gitlab::HTTP::Error => e
Rails.logger.info "Unable to contact GitLab, Inc.: #{e}" # rubocop:disable Gitlab/RailsLogger
Gitlab::AppLogger.info("Unable to contact GitLab, Inc.: #{e}")
false
end
......
......@@ -63,7 +63,7 @@ class WebHookService
error_message: e.to_s
)
Rails.logger.error("WebHook Error => #{e}") # rubocop:disable Gitlab/RailsLogger
Gitlab::AppLogger.error("WebHook Error => #{e}")
{
status: :error,
......
---
title: Use applogger in usage ping and webhook service
merge_request: 32192
author: Rajendra Kadam
type: fixed
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment