Merge pull request #2273 from Atralupus/feat/acs-logging

Add acs access denied debug log
planetarium · Oct 19, 2023 · e44aa4c · e44aa4c
2 parents dbdbd3c + 9c06624
commit e44aa4c
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 3 deletions.
diff --git a/NineChronicles.Headless/Services/RedisAccessControlService.cs b/NineChronicles.Headless/Services/RedisAccessControlService.cs
@@ -1,6 +1,7 @@
 using StackExchange.Redis;
 using Libplanet.Crypto;
 using Nekoyume.Blockchain;
+using Serilog;
 
 namespace NineChronicles.Headless.Services
 {
@@ -16,7 +17,13 @@ public RedisAccessControlService(string storageUri)
 
         public bool IsAccessDenied(Address address)
         {
-            return _db.KeyExists(address.ToString());
+            var result = _db.KeyExists(address.ToString());
+            if (result)
+            {
+                Log.Debug($"{address} is access denied");
+            }
+
+            return result;
         }
     }
 }
diff --git a/NineChronicles.Headless/Services/SQLiteAccessControlService.cs b/NineChronicles.Headless/Services/SQLiteAccessControlService.cs
@@ -1,6 +1,7 @@
 using Microsoft.Data.Sqlite;
 using Libplanet.Crypto;
 using Nekoyume.Blockchain;
+using Serilog;
 
 namespace NineChronicles.Headless.Services
 {
@@ -33,9 +34,16 @@ public bool IsAccessDenied(Address address)
             command.CommandText = CheckAccessSql;
             command.Parameters.AddWithValue("@Address", address.ToString());
 
-            var result = command.ExecuteScalar();
+            var queryResult = command.ExecuteScalar();
 
-            return result is not null && (long)result == 1;
+            var result = queryResult is not null && (long)queryResult == 1;
+
+            if (result)
+            {
+                Log.Debug($"{address} is access denied");
+            }
+
+            return result;
         }
     }
 }