From f0eff6131acd7bfb10a8453b419433b626de3312 Mon Sep 17 00:00:00 2001
From: Amir Hasanbasic <ahasanbasic@renderedtext.com>
Date: Mon, 23 Jun 2025 15:49:54 +0200
Subject: [PATCH] chore(hooks_processor): Do not log internal requests

---
 hooks_processor/lib/hooks_processor/clients/rbac_client.ex    | 2 +-
 hooks_processor/lib/hooks_processor/clients/user_client.ex    | 2 +-
 .../lib/hooks_processor/hooks/processing/git_worker.ex        | 4 ++--
 hooks_processor/lib/hooks_processor/hooks/processing/utils.ex | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/hooks_processor/lib/hooks_processor/clients/rbac_client.ex b/hooks_processor/lib/hooks_processor/clients/rbac_client.ex
index d6af6c159..5a79514b2 100644
--- a/hooks_processor/lib/hooks_processor/clients/rbac_client.ex
+++ b/hooks_processor/lib/hooks_processor/clients/rbac_client.ex
@@ -10,7 +10,7 @@ defmodule HooksProcessor.Clients.RBACClient do
   @grpc_timeout 5_000
 
   def member?(org_id, user_id) do
-    Logger.info("Calling RBAC API to check membership: org_id: #{org_id} user_id: #{user_id}")
+    Logger.debug("Calling RBAC API to check membership: org_id: #{org_id} user_id: #{user_id}")
 
     Metrics.benchmark("HooksProcessor.RBACClient", ["is_member?"], fn ->
       %ListUserPermissionsRequest{
diff --git a/hooks_processor/lib/hooks_processor/clients/user_client.ex b/hooks_processor/lib/hooks_processor/clients/user_client.ex
index 5b534a760..431fc4227 100644
--- a/hooks_processor/lib/hooks_processor/clients/user_client.ex
+++ b/hooks_processor/lib/hooks_processor/clients/user_client.ex
@@ -22,7 +22,7 @@ defmodule HooksProcessor.Clients.UserClient do
 
   def describe(user_id) do
     "user_id: #{user_id}"
-    |> LT.info("Calling User API to find requester")
+    |> LT.debug("Calling User API to find requester")
 
     Metrics.benchmark("HooksProcessor.UserClient", ["describe"], fn ->
       %DescribeRequest{
diff --git a/hooks_processor/lib/hooks_processor/hooks/processing/git_worker.ex b/hooks_processor/lib/hooks_processor/hooks/processing/git_worker.ex
index e84c76445..d0e6b04a7 100644
--- a/hooks_processor/lib/hooks_processor/hooks/processing/git_worker.ex
+++ b/hooks_processor/lib/hooks_processor/hooks/processing/git_worker.ex
@@ -55,7 +55,7 @@ defmodule HooksProcessor.Hooks.Processing.GitWorker do
 
   defp get_requester_id(webhook, email) do
     "email: #{email}"
-    |> LT.info("Hook #{webhook.id} - calling User API to find requester")
+    |> LT.debug("Hook #{webhook.id} - calling User API to find requester")
 
     case UserClient.describe_by_email(email) do
       {:ok, user} ->
@@ -73,7 +73,7 @@ defmodule HooksProcessor.Hooks.Processing.GitWorker do
 
   defp filter_membership(webhook, organization_id, user_id) do
     "organization_id: #{organization_id}, user_id: #{user_id}"
-    |> LT.info("Hook #{webhook.id} - calling RBAC API to check membership")
+    |> LT.debug("Hook #{webhook.id} - calling RBAC API to check membership")
 
     case RBACClient.member?(organization_id, user_id) do
       {:ok, true} ->
diff --git a/hooks_processor/lib/hooks_processor/hooks/processing/utils.ex b/hooks_processor/lib/hooks_processor/hooks/processing/utils.ex
index a9c617ff3..0f82a76ab 100644
--- a/hooks_processor/lib/hooks_processor/hooks/processing/utils.ex
+++ b/hooks_processor/lib/hooks_processor/hooks/processing/utils.ex
@@ -39,7 +39,7 @@ defmodule HooksProcessor.Hooks.Processing.Utils do
 
   def get_requester_id(webhook, provider_uid, provider_type) do
     "provider_uid: #{provider_uid} for provider: #{provider_type}"
-    |> LT.info("Hook #{webhook.id} - calling User API to find requester")
+    |> LT.debug("Hook #{webhook.id} - calling User API to find requester")
 
     case UserClient.describe_by_repository_provider(provider_uid, provider_type) do
       {:ok, user} ->