fix(deploy): tradein_fdw_reader password — psql var interpolation in DO block
psql :'pw' substitution НЕ работает внутри dollar-quoted блока $$..$$ — это правило psql, не bug. Symptom: deploy 2026-05-24 после merge PR #503 упал с 'syntax error at or near :' on LINE 4 of set_tradein_fdw_password.sql, :'pw' дошёл до сервера как literal вместо интерполяции. Fix: передаём password в DO через session GUC (set_config) — psql интерполирует :'pw' ВНЕ dollar quote, внутри читаем через current_setting(). Очищаем GUC после выполнения (defense-in-depth).
This commit is contained in:
parent
099b8e4b59
commit
da7814810c
1 changed files with 14 additions and 6 deletions
|
|
@ -10,18 +10,26 @@
|
||||||
-- Combined with the regex whitelist on the backend side (fdw.py _PASSWORD_RE)
|
-- Combined with the regex whitelist on the backend side (fdw.py _PASSWORD_RE)
|
||||||
-- this gives defense-in-depth.
|
-- this gives defense-in-depth.
|
||||||
--
|
--
|
||||||
-- Extracted from deploy.yml inline DO block (PR #493 deploy/1156 incident).
|
-- psql variable substitution (:'pw') НЕ интерполируется внутри dollar-quoted
|
||||||
-- The original block used multi-line backslash continuations inside a double-quoted
|
-- блока ($$...$$) — это правило psql, не bug. Поэтому password передаём в DO
|
||||||
-- bash string with $$-dollar-quoting + psql variable substitution — fragile escaping
|
-- через сессионный GUC (set_config), который psql интерполирует ВНЕ dollar
|
||||||
-- that likely caused the bootstrap step to fail before `compose up -d` ran.
|
-- quote, и читаем внутри через current_setting().
|
||||||
-- Using `-f <file>` with psql-native variable substitution via `-v pw=` is safe.
|
-- Reference incident: deploy 2026-05-24 (post-merge PR #503) упал на
|
||||||
|
-- "syntax error at or near ':'" в LINE 4 этого файла — :'pw' дошёл до сервера
|
||||||
|
-- as literal вместо интерполяции.
|
||||||
|
|
||||||
|
SELECT set_config('app.fdw_pw', :'pw', false);
|
||||||
|
|
||||||
DO $$
|
DO $$
|
||||||
BEGIN
|
BEGIN
|
||||||
IF EXISTS (SELECT 1 FROM pg_roles WHERE rolname = 'tradein_fdw_reader') THEN
|
IF EXISTS (SELECT 1 FROM pg_roles WHERE rolname = 'tradein_fdw_reader') THEN
|
||||||
EXECUTE format('ALTER ROLE tradein_fdw_reader WITH PASSWORD %L', :'pw');
|
EXECUTE format('ALTER ROLE tradein_fdw_reader WITH PASSWORD %L', current_setting('app.fdw_pw'));
|
||||||
RAISE NOTICE 'tradein_fdw_reader password set';
|
RAISE NOTICE 'tradein_fdw_reader password set';
|
||||||
ELSE
|
ELSE
|
||||||
RAISE NOTICE 'tradein_fdw_reader role missing — migration 100_tradein_fdw_role.sql not applied yet';
|
RAISE NOTICE 'tradein_fdw_reader role missing — migration 100_tradein_fdw_role.sql not applied yet';
|
||||||
END IF;
|
END IF;
|
||||||
END $$;
|
END $$;
|
||||||
|
|
||||||
|
-- Clear GUC after use (defense-in-depth — не оставляем password в session state
|
||||||
|
-- даже на short connection).
|
||||||
|
SELECT set_config('app.fdw_pw', '', false);
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue