pgsql: Be precise about when ALTER SUBSCRIPTION needs conninfo. - Mailing list pgsql-committers

From Jeff Davis
Subject pgsql: Be precise about when ALTER SUBSCRIPTION needs conninfo.
Date
Msg-id E1wri0A-00000000LC7-2fZz@gemulon.postgresql.org
Whole thread
List pgsql-committers
Be precise about when ALTER SUBSCRIPTION needs conninfo.

Decide early whether the original conninfo is needed so that errors
happen consistently.

Addresses finding 12 in report from linked discussion.

Co-authored-by: Shlok Kyal <shlok.kyal.oss@gmail.com>
Reported-by: Noah Misch <noah@leadboat.com>
Reviewed-by: Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com>
Reviewed-by: Amit Kapila <amit.kapila16@gmail.com>
Discussion: https://postgr.es/m/20260710195902.4f.noahmisch%40microsoft.com
Backpatch-through: 19

Branch
------
REL_19_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/ae420d66e72051b7ceab49d14f990094f1c8c6b4

Modified Files
--------------
src/backend/commands/subscriptioncmds.c    | 79 +++++++++++++++++++-----------
src/test/regress/expected/subscription.out |  8 +++
src/test/regress/sql/subscription.sql      |  9 ++++
3 files changed, 68 insertions(+), 28 deletions(-)


pgsql-committers by date:

Previous
From: Jeff Davis
Date:
Subject: pgsql: Remove Subscription conninfo field; generate in caller.
Next
From: Jeff Davis
Date:
Subject: pgsql: Revert "Validate subscription conninfo on owner change"