fix(logging): log at higher level for some auth failures (#463)

When a param is missing from the `hasPermission` call, log it louder
spinnaker · Sep 3, 2019 · 8cdc9fc · 8cdc9fc
1 parent beb8238
commit 8cdc9fc
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/fiat-api/src/main/java/com/netflix/spinnaker/fiat/shared/FiatPermissionEvaluator.java b/fiat-api/src/main/java/com/netflix/spinnaker/fiat/shared/FiatPermissionEvaluator.java
@@ -151,8 +151,8 @@ public boolean hasPermission(
       return true;
     }
     if (resourceName == null || resourceType == null || authorization == null) {
-      log.debug(
-          "Permission denied due to null argument. resourceName={}, resourceType={}, "
+      log.warn(
+          "Permission denied because at least one of the required arguments was null. resourceName={}, resourceType={}, "
               + "authorization={}",
           resourceName,
           resourceType,