Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Mon, 8 Apr 2024 at 10:29, Masahiko Sawada wrote: > > On Fri, Apr 5, 2024 at 1:18 AM vignesh C wrote: > > > > On Tue, 2 Apr 2024 at 13:08, Masahiko Sawada wrote: > > > > > > On Mon, Apr 1, 2024 at 10:41 PM vignesh C wrote: > > > > > > > > On Thu, 28 Mar 2024 at 13:05, Masahiko Sawada > > > > wrote: > > > > > > > > > > Hi, > > > > > > > > > > Thank you for the patch! > > > > > > > > > > On Mon, Jul 3, 2023 at 12:12 AM vignesh C wrote: > > > > > > > > > > > > Hi, > > > > > > > > > > > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER > > > > > > TABLE": > > > > > > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > > > > > > completion of alter default privileges like the below statement: > > > > > > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > > > > > > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > > > > > > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables > > > > > > FROM dba1; > > > > > > > > > > +1 > > > > > > > > > > > > > > > > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > > > > > > public FOR " like in below statement: > > > > > > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > > > > > > ON TABLES TO PUBLIC; > > > > > > > > > > Since there is no difference FOR USER and FOR ROLE, I'm not sure we > > > > > really want to support both in tab-completion. > > > > > > > > I have removed this change > > > > > > > > > > > > > > > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > > > > > > REVOKE " like in below statement: > > > > > > alter default privileges revoke grant option for select ON tables > > > > > > FROM dba1; > > > > > > > > > > +1. But the v3 patch doesn't cover the following case: > > > > > > > > > > =# alter default privileges for role masahiko revoke [tab] > > > > > ALL CREATE DELETE EXECUTE INSERT MAINTAIN > > > > > REFERENCES SELECT TRIGGER TRUNCATEUPDATE USAGE > > > > > > > > Modified in the updated patch > > > > > > > > > And it doesn't cover MAINTAIN neither: > > > > > > > > > > =# alter default privileges revoke [tab] > > > > > ALL DELETEGRANT OPTION FOR REFERENCES > > > > > TRIGGER UPDATE > > > > > CREATEEXECUTE INSERTSELECT > > > > > TRUNCATE USAGE > > > > > > > > Modified in the updated patch > > > > > > > > > The patch adds the completions for ALTER DEFAULT PRIVILEGES REVOKE, > > > > > but we handle such case in GRANT and REVOKE part: > > > > > > > > > > (around L3958) > > > > > /* > > > > > * With ALTER DEFAULT PRIVILEGES, restrict completion to > > > > > grantable > > > > > * privileges (can't grant roles) > > > > > */ > > > > > if (HeadMatches("ALTER", "DEFAULT", "PRIVILEGES")) > > > > > COMPLETE_WITH("SELECT", "INSERT", "UPDATE", > > > > > "DELETE", "TRUNCATE", "REFERENCES", > > > > > "TRIGGER", > > > > > "CREATE", "EXECUTE", "USAGE", "MAINTAIN", > > > > > "ALL"); > > > > > > > > The current patch handles the fix from here now. > > > > > > > > > Also, I think we can support WITH GRANT OPTION too. For example, > > > > > > > > > > =# alter default privileges for role masahiko grant all on tables to > > > > > public [tab] > > > > > > > > I have handled this in the updated patch > > > > > > > > > It's already supported in the GRANT statement. > > > > > > > > > > > > > > > > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > > > > > > column-name SET" like in: > > > > > > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > > > > > > > > > > > > > > > +1. The patch looks good to me, so pushed. > > > > > > > > Thanks for committing this. > > > > > > > > The updated patch has the changes for the above comments. > > > > > > > > > > Thank you for updating the patch. > > > > > > I think it doesn't work well as "GRANT OPTION FOR" is complemented > > > twice. For example, > > > > > > =# alter default privileges for user masahiko revoke [tab] > > > ALL DELETEGRANT OPTION FOR MAINTAIN > > > SELECTTRUNCATE USAGE > > > CREATEEXECUTE INSERTREFERENCES > > > TRIGGER UPDATE > > > =# alter default privileges for user masahiko revoke grant option for > > > [tab] > > > ALL DELETEGRANT OPTION FOR MAINTAIN > > > SELECTTRUNCATE USAGE > > > CREATEEXECUTE INSERTREFERENCES > > > TRIGGER UPDATE > > > > Thanks for finding this issue, the attached v5 version patch has the > > fix for the same. > > Thank you for updating the patch! I've pushed with minor adjustments. Thanks for pushing this patch. Regards, Vignesh
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Fri, Apr 5, 2024 at 1:18 AM vignesh C wrote: > > On Tue, 2 Apr 2024 at 13:08, Masahiko Sawada wrote: > > > > On Mon, Apr 1, 2024 at 10:41 PM vignesh C wrote: > > > > > > On Thu, 28 Mar 2024 at 13:05, Masahiko Sawada > > > wrote: > > > > > > > > Hi, > > > > > > > > Thank you for the patch! > > > > > > > > On Mon, Jul 3, 2023 at 12:12 AM vignesh C wrote: > > > > > > > > > > Hi, > > > > > > > > > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER > > > > > TABLE": > > > > > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > > > > > completion of alter default privileges like the below statement: > > > > > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > > > > > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > > > > > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables > > > > > FROM dba1; > > > > > > > > +1 > > > > > > > > > > > > > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > > > > > public FOR " like in below statement: > > > > > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > > > > > ON TABLES TO PUBLIC; > > > > > > > > Since there is no difference FOR USER and FOR ROLE, I'm not sure we > > > > really want to support both in tab-completion. > > > > > > I have removed this change > > > > > > > > > > > > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > > > > > REVOKE " like in below statement: > > > > > alter default privileges revoke grant option for select ON tables > > > > > FROM dba1; > > > > > > > > +1. But the v3 patch doesn't cover the following case: > > > > > > > > =# alter default privileges for role masahiko revoke [tab] > > > > ALL CREATE DELETE EXECUTE INSERT MAINTAIN > > > > REFERENCES SELECT TRIGGER TRUNCATEUPDATE USAGE > > > > > > Modified in the updated patch > > > > > > > And it doesn't cover MAINTAIN neither: > > > > > > > > =# alter default privileges revoke [tab] > > > > ALL DELETEGRANT OPTION FOR REFERENCES > > > > TRIGGER UPDATE > > > > CREATEEXECUTE INSERTSELECT > > > > TRUNCATE USAGE > > > > > > Modified in the updated patch > > > > > > > The patch adds the completions for ALTER DEFAULT PRIVILEGES REVOKE, > > > > but we handle such case in GRANT and REVOKE part: > > > > > > > > (around L3958) > > > > /* > > > > * With ALTER DEFAULT PRIVILEGES, restrict completion to > > > > grantable > > > > * privileges (can't grant roles) > > > > */ > > > > if (HeadMatches("ALTER", "DEFAULT", "PRIVILEGES")) > > > > COMPLETE_WITH("SELECT", "INSERT", "UPDATE", > > > > "DELETE", "TRUNCATE", "REFERENCES", "TRIGGER", > > > > "CREATE", "EXECUTE", "USAGE", "MAINTAIN", > > > > "ALL"); > > > > > > The current patch handles the fix from here now. > > > > > > > Also, I think we can support WITH GRANT OPTION too. For example, > > > > > > > > =# alter default privileges for role masahiko grant all on tables to > > > > public [tab] > > > > > > I have handled this in the updated patch > > > > > > > It's already supported in the GRANT statement. > > > > > > > > > > > > > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > > > > > column-name SET" like in: > > > > > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > > > > > > > > > > > > +1. The patch looks good to me, so pushed. > > > > > > Thanks for committing this. > > > > > > The updated patch has the changes for the above comments. > > > > > > > Thank you for updating the patch. > > > > I think it doesn't work well as "GRANT OPTION FOR" is complemented > > twice. For example, > > > > =# alter default privileges for user masahiko revoke [tab] > > ALL DELETEGRANT OPTION FOR MAINTAIN > > SELECTTRUNCATE USAGE > > CREATEEXECUTE INSERTREFERENCES > > TRIGGER UPDATE > > =# alter default privileges for user masahiko revoke grant option for [tab] > > ALL DELETEGRANT OPTION FOR MAINTAIN > > SELECTTRUNCATE USAGE > > CREATEEXECUTE INSERTREFERENCES > > TRIGGER UPDATE > > Thanks for finding this issue, the attached v5 version patch has the > fix for the same. Thank you for updating the patch! I've pushed with minor adjustments. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Tue, 2 Apr 2024 at 13:08, Masahiko Sawada wrote: > > On Mon, Apr 1, 2024 at 10:41 PM vignesh C wrote: > > > > On Thu, 28 Mar 2024 at 13:05, Masahiko Sawada wrote: > > > > > > Hi, > > > > > > Thank you for the patch! > > > > > > On Mon, Jul 3, 2023 at 12:12 AM vignesh C wrote: > > > > > > > > Hi, > > > > > > > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > > > > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > > > > completion of alter default privileges like the below statement: > > > > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > > > > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > > > > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM > > > > dba1; > > > > > > +1 > > > > > > > > > > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > > > > public FOR " like in below statement: > > > > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > > > > ON TABLES TO PUBLIC; > > > > > > Since there is no difference FOR USER and FOR ROLE, I'm not sure we > > > really want to support both in tab-completion. > > > > I have removed this change > > > > > > > > > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > > > > REVOKE " like in below statement: > > > > alter default privileges revoke grant option for select ON tables FROM > > > > dba1; > > > > > > +1. But the v3 patch doesn't cover the following case: > > > > > > =# alter default privileges for role masahiko revoke [tab] > > > ALL CREATE DELETE EXECUTE INSERT MAINTAIN > > > REFERENCES SELECT TRIGGER TRUNCATEUPDATE USAGE > > > > Modified in the updated patch > > > > > And it doesn't cover MAINTAIN neither: > > > > > > =# alter default privileges revoke [tab] > > > ALL DELETEGRANT OPTION FOR REFERENCES > > > TRIGGER UPDATE > > > CREATEEXECUTE INSERTSELECT > > > TRUNCATE USAGE > > > > Modified in the updated patch > > > > > The patch adds the completions for ALTER DEFAULT PRIVILEGES REVOKE, > > > but we handle such case in GRANT and REVOKE part: > > > > > > (around L3958) > > > /* > > > * With ALTER DEFAULT PRIVILEGES, restrict completion to grantable > > > * privileges (can't grant roles) > > > */ > > > if (HeadMatches("ALTER", "DEFAULT", "PRIVILEGES")) > > > COMPLETE_WITH("SELECT", "INSERT", "UPDATE", > > > "DELETE", "TRUNCATE", "REFERENCES", "TRIGGER", > > > "CREATE", "EXECUTE", "USAGE", "MAINTAIN", > > > "ALL"); > > > > The current patch handles the fix from here now. > > > > > Also, I think we can support WITH GRANT OPTION too. For example, > > > > > > =# alter default privileges for role masahiko grant all on tables to > > > public [tab] > > > > I have handled this in the updated patch > > > > > It's already supported in the GRANT statement. > > > > > > > > > > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > > > > column-name SET" like in: > > > > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > > > > > > > > > +1. The patch looks good to me, so pushed. > > > > Thanks for committing this. > > > > The updated patch has the changes for the above comments. > > > > Thank you for updating the patch. > > I think it doesn't work well as "GRANT OPTION FOR" is complemented > twice. For example, > > =# alter default privileges for user masahiko revoke [tab] > ALL DELETEGRANT OPTION FOR MAINTAIN > SELECTTRUNCATE USAGE > CREATEEXECUTE INSERTREFERENCES > TRIGGER UPDATE > =# alter default privileges for user masahiko revoke grant option for [tab] > ALL DELETEGRANT OPTION FOR MAINTAIN > SELECTTRUNCATE USAGE > CREATEEXECUTE INSERTREFERENCES > TRIGGER UPDATE Thanks for finding this issue, the attached v5 version patch has the fix for the same. Regards, Vignesh From 7aecc1d577f5608384044692dbffee7018e434fd Mon Sep 17 00:00:00 2001 From: Vignesh C Date: Sun, 2 Jul 2023 19:35:46 +0530 Subject: [PATCH v5] Fix missing tab completion in "ALTER DEFAULT PRIVILEGES" GRANT, REVOKE and FOR USER keyword was not displayed in tab completion of "ALTER DEFAULT PRIVILEGES" like the below statements: ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; ALTER DEFAULT PRIVILEGES FOR USER testdba REVOKE INSERT ON tables FROM testdba; "GRANT OPTION OPTION" was not displayed in tab completion of "ALTER DEFAULT PRIVILEGES REVOKE " like in below statement: ALTER DEFAULT PRIVILEGES REVOKE GRANT OPTION FOR SELECT ON tables FROM testdba; "WITH GRANT OPTION" was not completed in tab completion of "ALTER DEFAULT PRIVILEGES
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Mon, Apr 1, 2024 at 10:41 PM vignesh C wrote: > > On Thu, 28 Mar 2024 at 13:05, Masahiko Sawada wrote: > > > > Hi, > > > > Thank you for the patch! > > > > On Mon, Jul 3, 2023 at 12:12 AM vignesh C wrote: > > > > > > Hi, > > > > > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > > > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > > > completion of alter default privileges like the below statement: > > > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > > > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > > > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM > > > dba1; > > > > +1 > > > > > > > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > > > public FOR " like in below statement: > > > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > > > ON TABLES TO PUBLIC; > > > > Since there is no difference FOR USER and FOR ROLE, I'm not sure we > > really want to support both in tab-completion. > > I have removed this change > > > > > > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > > > REVOKE " like in below statement: > > > alter default privileges revoke grant option for select ON tables FROM > > > dba1; > > > > +1. But the v3 patch doesn't cover the following case: > > > > =# alter default privileges for role masahiko revoke [tab] > > ALL CREATE DELETE EXECUTE INSERT MAINTAIN > > REFERENCES SELECT TRIGGER TRUNCATEUPDATE USAGE > > Modified in the updated patch > > > And it doesn't cover MAINTAIN neither: > > > > =# alter default privileges revoke [tab] > > ALL DELETEGRANT OPTION FOR REFERENCES > > TRIGGER UPDATE > > CREATEEXECUTE INSERTSELECT > > TRUNCATE USAGE > > Modified in the updated patch > > > The patch adds the completions for ALTER DEFAULT PRIVILEGES REVOKE, > > but we handle such case in GRANT and REVOKE part: > > > > (around L3958) > > /* > > * With ALTER DEFAULT PRIVILEGES, restrict completion to grantable > > * privileges (can't grant roles) > > */ > > if (HeadMatches("ALTER", "DEFAULT", "PRIVILEGES")) > > COMPLETE_WITH("SELECT", "INSERT", "UPDATE", > > "DELETE", "TRUNCATE", "REFERENCES", "TRIGGER", > > "CREATE", "EXECUTE", "USAGE", "MAINTAIN", "ALL"); > > The current patch handles the fix from here now. > > > Also, I think we can support WITH GRANT OPTION too. For example, > > > > =# alter default privileges for role masahiko grant all on tables to > > public [tab] > > I have handled this in the updated patch > > > It's already supported in the GRANT statement. > > > > > > > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > > > column-name SET" like in: > > > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > > > > > > +1. The patch looks good to me, so pushed. > > Thanks for committing this. > > The updated patch has the changes for the above comments. > Thank you for updating the patch. I think it doesn't work well as "GRANT OPTION FOR" is complemented twice. For example, =# alter default privileges for user masahiko revoke [tab] ALL DELETEGRANT OPTION FOR MAINTAIN SELECTTRUNCATE USAGE CREATEEXECUTE INSERTREFERENCES TRIGGER UPDATE =# alter default privileges for user masahiko revoke grant option for [tab] ALL DELETEGRANT OPTION FOR MAINTAIN SELECTTRUNCATE USAGE CREATEEXECUTE INSERTREFERENCES TRIGGER UPDATE Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Thu, 28 Mar 2024 at 13:05, Masahiko Sawada wrote: > > Hi, > > Thank you for the patch! > > On Mon, Jul 3, 2023 at 12:12 AM vignesh C wrote: > > > > Hi, > > > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > > completion of alter default privileges like the below statement: > > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM dba1; > > +1 > > > > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > > public FOR " like in below statement: > > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > > ON TABLES TO PUBLIC; > > Since there is no difference FOR USER and FOR ROLE, I'm not sure we > really want to support both in tab-completion. I have removed this change > > > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > > REVOKE " like in below statement: > > alter default privileges revoke grant option for select ON tables FROM dba1; > > +1. But the v3 patch doesn't cover the following case: > > =# alter default privileges for role masahiko revoke [tab] > ALL CREATE DELETE EXECUTE INSERT MAINTAIN > REFERENCES SELECT TRIGGER TRUNCATEUPDATE USAGE Modified in the updated patch > And it doesn't cover MAINTAIN neither: > > =# alter default privileges revoke [tab] > ALL DELETEGRANT OPTION FOR REFERENCES > TRIGGER UPDATE > CREATEEXECUTE INSERTSELECT > TRUNCATE USAGE Modified in the updated patch > The patch adds the completions for ALTER DEFAULT PRIVILEGES REVOKE, > but we handle such case in GRANT and REVOKE part: > > (around L3958) > /* > * With ALTER DEFAULT PRIVILEGES, restrict completion to grantable > * privileges (can't grant roles) > */ > if (HeadMatches("ALTER", "DEFAULT", "PRIVILEGES")) > COMPLETE_WITH("SELECT", "INSERT", "UPDATE", > "DELETE", "TRUNCATE", "REFERENCES", "TRIGGER", > "CREATE", "EXECUTE", "USAGE", "MAINTAIN", "ALL"); The current patch handles the fix from here now. > Also, I think we can support WITH GRANT OPTION too. For example, > > =# alter default privileges for role masahiko grant all on tables to > public [tab] I have handled this in the updated patch > It's already supported in the GRANT statement. > > > > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > > column-name SET" like in: > > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > > > +1. The patch looks good to me, so pushed. Thanks for committing this. The updated patch has the changes for the above comments. Regards, Vignesh From 7c3bbfb0c8c17bf5d4a7e2ce579d00b811844a06 Mon Sep 17 00:00:00 2001 From: Vignesh C Date: Sun, 2 Jul 2023 19:35:46 +0530 Subject: [PATCH v4] Fix missing tab completion in "ALTER DEFAULT PRIVILEGES" GRANT, REVOKE and FOR USER keyword was not displayed in tab completion of "ALTER DEFAULT PRIVILEGES" like the below statements: ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; ALTER DEFAULT PRIVILEGES FOR USER testdba REVOKE INSERT ON tables FROM testdba; "GRANT OPTION OPTION" was not displayed in tab completion of "ALTER DEFAULT PRIVILEGES REVOKE " like in below statement: ALTER DEFAULT PRIVILEGES REVOKE GRANT OPTION FOR SELECT ON tables FROM testdba; "WITH GRANT OPTION" was not completed in tab completion of "ALTER DEFAULT PRIVILEGES ... GRANT ... to role" like in below statement: ALTER DEFAULT PRIVILEGES FOR ROLE testdba1 GRANT ALL ON tables to testdba2 WITH GRANT OPTION; --- src/bin/psql/tab-complete.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index 82eb3955ab..620628948a 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -2147,7 +2147,7 @@ psql_completion(const char *text, int start, int end) /* ALTER DEFAULT PRIVILEGES */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES")) - COMPLETE_WITH("FOR ROLE", "IN SCHEMA"); + COMPLETE_WITH("FOR", "GRANT", "IN SCHEMA", "REVOKE"); /* ALTER DEFAULT PRIVILEGES FOR */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "FOR")) COMPLETE_WITH("ROLE"); @@ -3949,9 +3949,17 @@ psql_completion(const char *text, int start, int end) * privileges (can't grant roles) */ if (HeadMatches("ALTER", "DEFAULT", "PRIVILEGES")) - COMPLETE_WITH("SELECT", "INSERT", "UPDATE", - "DELETE", "TRUNCATE", "REFERENCES", "TRIGGER", - "CREATE", "EXECUTE", "USAGE", "MAINTAIN", "ALL"); + { + if (TailMatches("GRANT")) +COMPLETE_WITH("SELECT", "INSERT", "UPDATE", +
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
Hi, Thank you for the patch! On Mon, Jul 3, 2023 at 12:12 AM vignesh C wrote: > > Hi, > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > completion of alter default privileges like the below statement: > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM dba1; +1 > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > public FOR " like in below statement: > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > ON TABLES TO PUBLIC; Since there is no difference FOR USER and FOR ROLE, I'm not sure we really want to support both in tab-completion. > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > REVOKE " like in below statement: > alter default privileges revoke grant option for select ON tables FROM dba1; +1. But the v3 patch doesn't cover the following case: =# alter default privileges for role masahiko revoke [tab] ALL CREATE DELETE EXECUTE INSERT MAINTAIN REFERENCES SELECT TRIGGER TRUNCATEUPDATE USAGE And it doesn't cover MAINTAIN neither: =# alter default privileges revoke [tab] ALL DELETEGRANT OPTION FOR REFERENCES TRIGGER UPDATE CREATEEXECUTE INSERTSELECT TRUNCATE USAGE The patch adds the completions for ALTER DEFAULT PRIVILEGES REVOKE, but we handle such case in GRANT and REVOKE part: (around L3958) /* * With ALTER DEFAULT PRIVILEGES, restrict completion to grantable * privileges (can't grant roles) */ if (HeadMatches("ALTER", "DEFAULT", "PRIVILEGES")) COMPLETE_WITH("SELECT", "INSERT", "UPDATE", "DELETE", "TRUNCATE", "REFERENCES", "TRIGGER", "CREATE", "EXECUTE", "USAGE", "MAINTAIN", "ALL"); Also, I think we can support WITH GRANT OPTION too. For example, =# alter default privileges for role masahiko grant all on tables to public [tab] It's already supported in the GRANT statement. > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > column-name SET" like in: > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > +1. The patch looks good to me, so pushed. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Mon, 27 Nov 2023 at 21:58, vignesh C wrote: > > On Fri, 24 Nov 2023 at 18:37, Shubham Khanna > wrote: > > > > n Fri, Nov 24, 2023 at 6:33 PM vignesh C wrote: > > > > > > Hi, > > > > > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > > > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > > > completion of alter default privileges like the below statement: > > > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > > > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > > > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM > > > dba1; > > > > > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > > > public FOR " like in below statement: > > > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > > > ON TABLES TO PUBLIC; > > > > > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > > > REVOKE " like in below statement: > > > alter default privileges revoke grant option for select ON tables FROM > > > dba1; > > > > > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > > > column-name SET" like in: > > > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > > > > > Attached patch has the changes for the same. > > > > +COMPLETE_WITH("ROLE", "USER"); > > + /* ALTER DEFAULT PRIVILEGES REVOKE */ > > + else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "REVOKE")) > > +COMPLETE_WITH("SELECT", "INSERT", "UPDATE", "DELETE", "TRUNCATE", > > + "REFERENCES", "TRIGGER", "CREATE", "EXECUTE", "USAGE", > > + "MAINTAIN", "ALL", "GRANT OPTION FOR"); > > > > I could not find "alter default privileges revoke maintain", should > > this be removed? > > This was reverted as part of: > 151c22deee66a3390ca9a1c3675e29de54ae73fc. > Revert MAINTAIN privilege and pg_maintain predefined role. > > This reverts the following commits: 4dbdb82513, c2122aae63, > 5b1a879943, 9e1e9d6560, ff9618e82a, 60684dd834, 4441fc704d, > and b5d6382496. A role with the MAINTAIN privilege may be able to > use search_path tricks to escalate privileges to the table owner. > Unfortunately, it is too late in the v16 development cycle to apply > the proposed fix, i.e., restricting search_path when running > maintenance commands. > > The attached v2 version has the changes for the same. The patch was not applying because of a recent commit in tab completion, PSA new patch set. Regards, Vignesh From e3695181cf7d5f31dda520a01def544dc576d6c1 Mon Sep 17 00:00:00 2001 From: Vignesh C Date: Sun, 2 Jul 2023 19:35:46 +0530 Subject: [PATCH v3 1/2] Fix missing tab completion in "ALTER DEFAULT PRIVILEGES" GRANT, REVOKE and FOR USER keyword was not displayed in tab completion of alter default prvileges like the below statement: ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM vignesh; USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR " like in below statement: ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER vignesh GRANT INSERT ON TABLES TO PUBLIC; "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES REVOKE " like in below statement: alter default privileges revoke grant option for select ON tables FROM testdba; --- src/bin/psql/tab-complete.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index ada711d02f..d5b3794fd6 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -2144,10 +2144,15 @@ psql_completion(const char *text, int start, int end) /* ALTER DEFAULT PRIVILEGES */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES")) - COMPLETE_WITH("FOR ROLE", "IN SCHEMA"); + COMPLETE_WITH("FOR", "GRANT", "IN SCHEMA", "REVOKE"); /* ALTER DEFAULT PRIVILEGES FOR */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "FOR")) - COMPLETE_WITH("ROLE"); + COMPLETE_WITH("ROLE", "USER"); + /* ALTER DEFAULT PRIVILEGES REVOKE */ + else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "REVOKE")) + COMPLETE_WITH("SELECT", "INSERT", "UPDATE", "DELETE", "TRUNCATE", + "REFERENCES", "TRIGGER", "CREATE", "EXECUTE", "USAGE", + "ALL", "GRANT OPTION FOR"); /* ALTER DEFAULT PRIVILEGES IN */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "IN")) COMPLETE_WITH("SCHEMA"); @@ -2158,11 +2163,11 @@ psql_completion(const char *text, int start, int end) /* ALTER DEFAULT PRIVILEGES IN SCHEMA ... */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "IN", "SCHEMA", MatchAny)) - COMPLETE_WITH("GRANT", "REVOKE", "FOR ROLE"); + COMPLETE_WITH("GRANT", "REVOKE", "FOR"); /* ALTER DEFAULT PRIVILEGES IN SCHEMA ... FOR */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "IN", "SCHEMA", MatchAny, "FOR")) - COMPLETE_WITH("ROLE"); + COMPLETE_WITH("ROLE", "USER"); /*
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Mon, Nov 27, 2023 at 9:58 PM vignesh C wrote: > > On Fri, 24 Nov 2023 at 18:37, Shubham Khanna > wrote: > > > > n Fri, Nov 24, 2023 at 6:33 PM vignesh C wrote: > > > > > > Hi, > > > > > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > > > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > > > completion of alter default privileges like the below statement: > > > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > > > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > > > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM > > > dba1; > > > > > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > > > public FOR " like in below statement: > > > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > > > ON TABLES TO PUBLIC; > > > > > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > > > REVOKE " like in below statement: > > > alter default privileges revoke grant option for select ON tables FROM > > > dba1; > > > > > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > > > column-name SET" like in: > > > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > > > > > Attached patch has the changes for the same. > > > > +COMPLETE_WITH("ROLE", "USER"); > > + /* ALTER DEFAULT PRIVILEGES REVOKE */ > > + else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "REVOKE")) > > +COMPLETE_WITH("SELECT", "INSERT", "UPDATE", "DELETE", "TRUNCATE", > > + "REFERENCES", "TRIGGER", "CREATE", "EXECUTE", "USAGE", > > + "MAINTAIN", "ALL", "GRANT OPTION FOR"); > > > > I could not find "alter default privileges revoke maintain", should > > this be removed? > > This was reverted as part of: > 151c22deee66a3390ca9a1c3675e29de54ae73fc. > Revert MAINTAIN privilege and pg_maintain predefined role. > > This reverts the following commits: 4dbdb82513, c2122aae63, > 5b1a879943, 9e1e9d6560, ff9618e82a, 60684dd834, 4441fc704d, > and b5d6382496. A role with the MAINTAIN privilege may be able to > use search_path tricks to escalate privileges to the table owner. > Unfortunately, it is too late in the v16 development cycle to apply > the proposed fix, i.e., restricting search_path when running > maintenance commands. > I have executed the given changes and they are working fine. Thanks and Regards, Shubham Khanna.
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Fri, 24 Nov 2023 at 18:37, Shubham Khanna wrote: > > n Fri, Nov 24, 2023 at 6:33 PM vignesh C wrote: > > > > Hi, > > > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > > completion of alter default privileges like the below statement: > > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM dba1; > > > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > > public FOR " like in below statement: > > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > > ON TABLES TO PUBLIC; > > > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > > REVOKE " like in below statement: > > alter default privileges revoke grant option for select ON tables FROM dba1; > > > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > > column-name SET" like in: > > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > > > Attached patch has the changes for the same. > > +COMPLETE_WITH("ROLE", "USER"); > + /* ALTER DEFAULT PRIVILEGES REVOKE */ > + else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "REVOKE")) > +COMPLETE_WITH("SELECT", "INSERT", "UPDATE", "DELETE", "TRUNCATE", > + "REFERENCES", "TRIGGER", "CREATE", "EXECUTE", "USAGE", > + "MAINTAIN", "ALL", "GRANT OPTION FOR"); > > I could not find "alter default privileges revoke maintain", should > this be removed? This was reverted as part of: 151c22deee66a3390ca9a1c3675e29de54ae73fc. Revert MAINTAIN privilege and pg_maintain predefined role. This reverts the following commits: 4dbdb82513, c2122aae63, 5b1a879943, 9e1e9d6560, ff9618e82a, 60684dd834, 4441fc704d, and b5d6382496. A role with the MAINTAIN privilege may be able to use search_path tricks to escalate privileges to the table owner. Unfortunately, it is too late in the v16 development cycle to apply the proposed fix, i.e., restricting search_path when running maintenance commands. The attached v2 version has the changes for the same. Regards, Vignesh From 511c14a51ba012400b8b849813d16c0a591666d0 Mon Sep 17 00:00:00 2001 From: Vignesh C Date: Sun, 2 Jul 2023 19:53:50 +0530 Subject: [PATCH v2 2/2] Fix missing tab completion in "ALTER TABLE table-name ALTER COLUMN column-name SET" "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN column-name SET" lke in: ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; --- src/bin/psql/tab-complete.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index 8fd639d102..737d253805 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -2488,7 +2488,7 @@ psql_completion(const char *text, int start, int end) /* ALTER TABLE ALTER [COLUMN] SET */ else if (Matches("ALTER", "TABLE", MatchAny, "ALTER", "COLUMN", MatchAny, "SET") || Matches("ALTER", "TABLE", MatchAny, "ALTER", MatchAny, "SET")) - COMPLETE_WITH("(", "COMPRESSION", "DEFAULT", "GENERATED", "NOT NULL", "STATISTICS", "STORAGE", + COMPLETE_WITH("(", "COMPRESSION", "DATA TYPE", "DEFAULT", "GENERATED", "NOT NULL", "STATISTICS", "STORAGE", /* a subset of ALTER SEQUENCE options */ "INCREMENT", "MINVALUE", "MAXVALUE", "START", "NO", "CACHE", "CYCLE"); /* ALTER TABLE ALTER [COLUMN] SET ( */ -- 2.34.1 From f5839d1eadba5019e760e361e545c4a778774a3e Mon Sep 17 00:00:00 2001 From: Vignesh C Date: Sun, 2 Jul 2023 19:35:46 +0530 Subject: [PATCH v2 1/2] Fix missing tab completion in "ALTER DEFAULT PRIVILEGES" GRANT, REVOKE and FOR USER keyword was not displayed in tab completion of alter default prvileges like the below statement: ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM vignesh; USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR " like in below statement: ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER vignesh GRANT INSERT ON TABLES TO PUBLIC; "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES REVOKE " like in below statement: alter default privileges revoke grant option for select ON tables FROM testdba; --- src/bin/psql/tab-complete.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index 006e10f5d2..8fd639d102 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -2137,10 +2137,15 @@ psql_completion(const char *text, int start, int end) /* ALTER DEFAULT PRIVILEGES */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES")) - COMPLETE_WITH("FOR ROLE", "IN SCHEMA"); + COMPLETE_WITH("FOR", "GRANT", "IN SCHEMA", "REVOKE"); /* ALTER DEFAULT PRIVILEGES FOR */
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
n Fri, Nov 24, 2023 at 6:33 PM vignesh C wrote: > > Hi, > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > completion of alter default privileges like the below statement: > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM dba1; > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > public FOR " like in below statement: > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > ON TABLES TO PUBLIC; > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > REVOKE " like in below statement: > alter default privileges revoke grant option for select ON tables FROM dba1; > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > column-name SET" like in: > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > Attached patch has the changes for the same. +COMPLETE_WITH("ROLE", "USER"); + /* ALTER DEFAULT PRIVILEGES REVOKE */ + else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "REVOKE")) +COMPLETE_WITH("SELECT", "INSERT", "UPDATE", "DELETE", "TRUNCATE", + "REFERENCES", "TRIGGER", "CREATE", "EXECUTE", "USAGE", + "MAINTAIN", "ALL", "GRANT OPTION FOR"); I could not find "alter default privileges revoke maintain", should this be removed? Regards, Shubham Khanna
Re: Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
On Sun, 2 Jul 2023 at 20:42, vignesh C wrote: > > Hi, > > Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": > 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab > completion of alter default privileges like the below statement: > ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; > ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; > ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM dba1; > > 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA > public FOR " like in below statement: > ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT > ON TABLES TO PUBLIC; > > 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES > REVOKE " like in below statement: > alter default privileges revoke grant option for select ON tables FROM dba1; > > 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN > column-name SET" like in: > ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; > > Attached patch has the changes for the same. Added a commitfest entry for this: https://commitfest.postgresql.org/45/4587/ Regards, Vignesh
Improve tab completion for ALTER DEFAULT PRIVILEGE and ALTER TABLE
Hi, Improved tab completion for "ALTER DEFAULT PRIVILEGE" and "ALTER TABLE": 1) GRANT, REVOKE and FOR USER keyword was not displayed in tab completion of alter default privileges like the below statement: ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM dba1; 2) USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR " like in below statement: ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER dba1 GRANT INSERT ON TABLES TO PUBLIC; 3) "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES REVOKE " like in below statement: alter default privileges revoke grant option for select ON tables FROM dba1; 4) "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN column-name SET" like in: ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; Attached patch has the changes for the same. Regards, Vignesh From de80163a02d5d528402b3547bec09f5207ead9d2 Mon Sep 17 00:00:00 2001 From: Vignesh C Date: Sun, 2 Jul 2023 19:35:46 +0530 Subject: [PATCH 1/2] Fix missing tab completion in "ALTER DEFAULT PRIVILEGES" GRANT, REVOKE and FOR USER keyword was not displayed in tab completion of alter default prvileges like the below statement: ALTER DEFAULT PRIVILEGES GRANT INSERT ON tables TO PUBLIC; ALTER DEFAULT PRIVILEGES REVOKE INSERT ON tables FROM PUBLIC; ALTER DEFAULT PRIVILEGES FOR USER vignesh revoke INSERT ON tables FROM vignesh; USER was not displayed for "ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR " like in below statement: ALTER DEFAULT PRIVILEGES IN SCHEMA public FOR USER vignesh GRANT INSERT ON TABLES TO PUBLIC; "FOR GRANT OPTION" was not display for "ALTER DEFAULT PRIVILEGES REVOKE " like in below statement: alter default privileges revoke grant option for select ON tables FROM testdba; --- src/bin/psql/tab-complete.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index 935bb9bd95..73502b0b95 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -2137,10 +2137,15 @@ psql_completion(const char *text, int start, int end) /* ALTER DEFAULT PRIVILEGES */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES")) - COMPLETE_WITH("FOR ROLE", "IN SCHEMA"); + COMPLETE_WITH("FOR", "GRANT", "IN SCHEMA", "REVOKE"); /* ALTER DEFAULT PRIVILEGES FOR */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "FOR")) - COMPLETE_WITH("ROLE"); + COMPLETE_WITH("ROLE", "USER"); + /* ALTER DEFAULT PRIVILEGES REVOKE */ + else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "REVOKE")) + COMPLETE_WITH("SELECT", "INSERT", "UPDATE", "DELETE", "TRUNCATE", + "REFERENCES", "TRIGGER", "CREATE", "EXECUTE", "USAGE", + "MAINTAIN", "ALL", "GRANT OPTION FOR"); /* ALTER DEFAULT PRIVILEGES IN */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "IN")) COMPLETE_WITH("SCHEMA"); @@ -2151,11 +2156,11 @@ psql_completion(const char *text, int start, int end) /* ALTER DEFAULT PRIVILEGES IN SCHEMA ... */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "IN", "SCHEMA", MatchAny)) - COMPLETE_WITH("GRANT", "REVOKE", "FOR ROLE"); + COMPLETE_WITH("GRANT", "REVOKE", "FOR"); /* ALTER DEFAULT PRIVILEGES IN SCHEMA ... FOR */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "IN", "SCHEMA", MatchAny, "FOR")) - COMPLETE_WITH("ROLE"); + COMPLETE_WITH("ROLE", "USER"); /* ALTER DEFAULT PRIVILEGES FOR ROLE|USER ... IN SCHEMA ... */ /* ALTER DEFAULT PRIVILEGES IN SCHEMA ... FOR ROLE|USER ... */ else if (Matches("ALTER", "DEFAULT", "PRIVILEGES", "FOR", "ROLE|USER", -- 2.34.1 From 7ff5cd41e5c1a33bcb91e1d32537556cb9808923 Mon Sep 17 00:00:00 2001 From: Vignesh C Date: Sun, 2 Jul 2023 19:53:50 +0530 Subject: [PATCH 2/2] Fix missing tab completion in "ALTER TABLE table-name ALTER COLUMN column-name SET" "DATA TYPE" was missing in "ALTER TABLE table-name ALTER COLUMN column-name SET" lke in: ALTER TABLE t1 ALTER COLUMN c1 SET DATA TYPE text; --- src/bin/psql/tab-complete.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index 73502b0b95..c71235462a 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -2488,7 +2488,7 @@ psql_completion(const char *text, int start, int end) /* ALTER TABLE ALTER [COLUMN] SET */ else if (Matches("ALTER", "TABLE", MatchAny, "ALTER", "COLUMN", MatchAny, "SET") || Matches("ALTER", "TABLE", MatchAny, "ALTER", MatchAny, "SET")) - COMPLETE_WITH("(", "COMPRESSION", "DEFAULT", "GENERATED", "NOT NULL", "STATISTICS", "STORAGE", + COMPLETE_WITH("(", "COMPRESSION", "DATA TYPE", "DEFAULT", "GENERATED", "NOT NULL", "STATISTICS", "STORAGE", /* a subset of ALTER SEQUENCE options */ "INCREMENT", "MINVALUE", "MAXVALUE", "START", "NO", "CACHE", "CYCLE"); /* ALTER TABLE ALTER [COLUMN]