postgres.git / summary / log / commit / refs

commit    88914332eaed702afb2cdbb0b776822738d58ece
Author:   Tomas Vondra <tomas.vondra@postgresql.org>
Date:     Thu Jul 31 13:17:26 2025 +0000
First released in: 18.0

    Fix tab completion for ALTER ROLE|USER ... RESET
    
    Commit c407d5426b87 added tab completion for ALTER ROLE|USER ... RESET,
    with the intent to offer only the variables actually set on the role.
    But as soon as the user started typing something, it would start to
    offer all possible matching variables.
    
    Fix this the same way ALTER DATABASE ... RESET does it, i.e. by
    properly considering the prefix.
    
    A second issue causing similar symptoms (offering variables that are not
    actually set for a role) was caused by a match to another pattern. The
    ALTER DATABASE ... RESET was already excluded, so do the same thing for
    ROLE/USER.
    
    Report and fix by Dagfinn Ilmari Mannsåker. Backpatch to 18, same as
    c407d5426b87.
    
    Author: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
    Reviewed-by: jian he <jian.universality@gmail.com>
    Discussion: https://postgr.es/m/87qzyghw2x.fsf%40wibble.ilmari.org
    Discussion: https://postgr.es/m/87tt4lumqz.fsf%40wibble.ilmari.org
    Backpatch-through: 18


 src/bin/psql/tab-complete.in.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/bin/psql/tab-complete.in.c b/src/bin/psql/tab-complete.in.c
index 39bf3024ed4..65cd6ec109b 100644
--- a/src/bin/psql/tab-complete.in.c
+++ b/src/bin/psql/tab-complete.in.c
@@ -1078,9 +1078,12 @@ Keywords_for_list_of_owner_roles, "PUBLIC"
 "  WHERE usename LIKE '%s'"
 
 #define Query_for_list_of_user_vars \
-" SELECT pg_catalog.split_part(pg_catalog.unnest(rolconfig),'=',1) "\
-"   FROM pg_catalog.pg_roles "\
-"  WHERE rolname LIKE '%s'"
+"SELECT conf FROM ("\
+"       SELECT rolname, pg_catalog.split_part(pg_catalog.unnest(rolconfig),'=',1) conf"\
+"         FROM pg_catalog.pg_roles"\
+"       ) s"\
+"  WHERE s.conf like '%s' "\
+"    AND s.rolname LIKE '%s'"
 
 #define Query_for_list_of_access_methods \
 " SELECT amname "\
@@ -2508,7 +2511,10 @@ match_previous_words(int pattern_id,
 
 	/* ALTER USER,ROLE <name> RESET */
 	else if (Matches("ALTER", "USER|ROLE", MatchAny, "RESET"))
+	{
+		set_completion_reference(prev2_wd);
 		COMPLETE_WITH_QUERY_PLUS(Query_for_list_of_user_vars, "ALL");
+	}
 
 	/* ALTER USER,ROLE <name> WITH */
 	else if (Matches("ALTER", "USER|ROLE", MatchAny, "WITH"))
@@ -4958,7 +4964,7 @@ match_previous_words(int pattern_id,
 	/* Complete with a variable name */
 	else if (TailMatches("SET|RESET") &&
 			 !TailMatches("UPDATE", MatchAny, "SET") &&
-			 !TailMatches("ALTER", "DATABASE", MatchAny, "RESET"))
+			 !TailMatches("ALTER", "DATABASE|USER|ROLE", MatchAny, "RESET"))
 		COMPLETE_WITH_QUERY_VERBATIM_PLUS(Query_for_list_of_set_vars,
 										  "CONSTRAINTS",
 										  "TRANSACTION",


[parent: 72c437f6e464]