Skip to content

Commit 32a6039

Browse files
committed
fix: update comments for noupdate() function access and clarify its role
1 parent 27b3dd0 commit 32a6039

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

supabase/migrations/20260104000000_revoke_process_function_queue_public_access.sql

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,8 +116,11 @@ REVOKE ALL ON FUNCTION "public"."generate_org_on_user_create"() FROM "authentica
116116
REVOKE ALL ON FUNCTION "public"."generate_org_user_on_org_create"() FROM "anon";
117117
REVOKE ALL ON FUNCTION "public"."generate_org_user_on_org_create"() FROM "authenticated";
118118

119-
REVOKE ALL ON FUNCTION "public"."noupdate"() FROM "anon";
120-
REVOKE ALL ON FUNCTION "public"."noupdate"() FROM "authenticated";
119+
-- NOTE: noupdate() is a trigger function used on the channels table.
120+
-- Users need EXECUTE permission on trigger functions to perform table operations.
121+
-- Revoking access would break channel updates for authenticated users.
122+
-- REVOKE ALL ON FUNCTION "public"."noupdate"() FROM "anon";
123+
-- REVOKE ALL ON FUNCTION "public"."noupdate"() FROM "authenticated";
121124

122125
REVOKE ALL ON FUNCTION "public"."record_deployment_history"() FROM "anon";
123126
REVOKE ALL ON FUNCTION "public"."record_deployment_history"() FROM "authenticated";

0 commit comments

Comments
 (0)