Skip to content

Commit 407a40a

Browse files
chore(hooks_processor): Do not log internal requests (#411)
## πŸ“ Description A bit more cutting of unimportant logging ## βœ… Checklist - [ ] I have tested this change - [ ] This change requires documentation update
1 parent 0ec35a1 commit 407a40a

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

β€Žhooks_processor/lib/hooks_processor/clients/rbac_client.ex

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ defmodule HooksProcessor.Clients.RBACClient do
1010
@grpc_timeout 5_000
1111

1212
def member?(org_id, user_id) do
13-
Logger.info("Calling RBAC API to check membership: org_id: #{org_id} user_id: #{user_id}")
13+
Logger.debug("Calling RBAC API to check membership: org_id: #{org_id} user_id: #{user_id}")
1414

1515
Metrics.benchmark("HooksProcessor.RBACClient", ["is_member?"], fn ->
1616
%ListUserPermissionsRequest{

β€Žhooks_processor/lib/hooks_processor/clients/user_client.ex

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ defmodule HooksProcessor.Clients.UserClient do
2222

2323
def describe(user_id) do
2424
"user_id: #{user_id}"
25-
|> LT.info("Calling User API to find requester")
25+
|> LT.debug("Calling User API to find requester")
2626

2727
Metrics.benchmark("HooksProcessor.UserClient", ["describe"], fn ->
2828
%DescribeRequest{

β€Žhooks_processor/lib/hooks_processor/hooks/processing/git_worker.ex

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ defmodule HooksProcessor.Hooks.Processing.GitWorker do
5555

5656
defp get_requester_id(webhook, email) do
5757
"email: #{email}"
58-
|> LT.info("Hook #{webhook.id} - calling User API to find requester")
58+
|> LT.debug("Hook #{webhook.id} - calling User API to find requester")
5959

6060
case UserClient.describe_by_email(email) do
6161
{:ok, user} ->
@@ -73,7 +73,7 @@ defmodule HooksProcessor.Hooks.Processing.GitWorker do
7373

7474
defp filter_membership(webhook, organization_id, user_id) do
7575
"organization_id: #{organization_id}, user_id: #{user_id}"
76-
|> LT.info("Hook #{webhook.id} - calling RBAC API to check membership")
76+
|> LT.debug("Hook #{webhook.id} - calling RBAC API to check membership")
7777

7878
case RBACClient.member?(organization_id, user_id) do
7979
{:ok, true} ->

β€Žhooks_processor/lib/hooks_processor/hooks/processing/utils.ex

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ defmodule HooksProcessor.Hooks.Processing.Utils do
3939

4040
def get_requester_id(webhook, provider_uid, provider_type) do
4141
"provider_uid: #{provider_uid} for provider: #{provider_type}"
42-
|> LT.info("Hook #{webhook.id} - calling User API to find requester")
42+
|> LT.debug("Hook #{webhook.id} - calling User API to find requester")
4343

4444
case UserClient.describe_by_repository_provider(provider_uid, provider_type) do
4545
{:ok, user} ->

0 commit comments

Comments
Β (0)