Skip to content

Commit e7101d2

Browse files
committed
chore: clean up rm comment will document with new issue
1 parent f8d9d85 commit e7101d2

2 files changed

Lines changed: 0 additions & 227 deletions

File tree

ansible/files/postgresql_extension_custom_scripts/amcheck/after-create.sql

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,3 @@
1-
-- Grant EXECUTE on amcheck's functions to the postgres role.
2-
--
3-
-- amcheck is pinned to pg_catalog. supautils creates it as supabase_admin, and neither
4-
-- pg_catalog's default privileges nor amcheck's own SQL (REVOKE ALL FROM PUBLIC) grant the
5-
-- customer anything, so without this the non-superuser postgres role cannot call
6-
-- bt_index_check(). Only postgres is granted; the API roles (anon, authenticated,
7-
-- service_role) are never granted, so amcheck stays off the REST surface. Granting
8-
-- explicitly here (rather than via a schema's default privileges) keeps amcheck's object
9-
-- privileges centrally managed in this repo. See PSQL-1327.
10-
--
11-
-- KNOWN LIMITATION: this runs only at CREATE. supautils has no after-update hook, so
12-
-- functions added by ALTER EXTENSION amcheck UPDATE (amcheck 1.3 -> 1.4 during a 15 -> 17
13-
-- pg_upgrade) are NOT granted to postgres. Accepted for now and tracked as a post-incident
14-
-- follow-up (e.g. an after-update hook in supautils, or disallowing extension-version
15-
-- selection so no in-place update is needed). The amcheck-upgrade VM test documents this.
161
do $$
172
declare
183
saved_search_path text := (select current_setting('search_path'));

nix/ext/tests/amcheck-upgrade.nix

Lines changed: 0 additions & 212 deletions
This file was deleted.

0 commit comments

Comments
 (0)