log a warning when paths are not allowed (#3316)

log a warning when paths are not allowed.
OSC · Jan 25, 2024 · 7ac4846 · 7ac4846
1 parent 2cd4a67
commit 7ac4846
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/apps/dashboard/app/models/allowlist_policy.rb b/apps/dashboard/app/models/allowlist_policy.rb
@@ -28,7 +28,11 @@ def permitted?(path)
   # @raises ArgumentError if any allowlist path or permitted? argument
   #         has the form ~user/some/path where user doesn't exist
   def validate!(path)
-    raise AllowlistPolicy::Forbidden, "#{path} does not have an ancestor directory specified in ALLOWLIST_PATH" unless permitted?(path.to_s)
+    return if permitted?(path.to_s)
+
+    msg = "#{path} does not have an ancestor directory specified in ALLOWLIST_PATH"
+    Rails.logger.warn(msg)
+    raise AllowlistPolicy::Forbidden, msg
   end
 
   protected