Re: PATCH: Login/Group Role Node - Mailing list pgadmin-hackers

From Dave Page
Subject Re: PATCH: Login/Group Role Node
Date
Msg-id CA+OCxoytmkdf3a-ngUz1r2b_7OU1aC3bCv48T=hofQKM-PYt=A@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: Login/Group Role Node  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
List pgadmin-hackers


On Tue, Feb 2, 2016 at 9:22 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Ashesh,

I also found that we are not handling group role in macro templates.

Steps to re-produce the issue:
1) Create a sample Group Role.
    name : my_group

Click 'Ok' to create.

2) Go to privilege tab (in my case Database node)
- Click on Add button
- Select my_group (Group) from options 
- grant any priviledges

3) Go to modified SQL tab

- You will see wrong SQL generated
GRANT ALL ON DATABASE postgres TO my_group WITH GRANT OPTIONS

Expected SQL:
GRANT ALL ON DATABASE postgres TO GROUP my_group WITH GRANT OPTIONS

Isn't GROUP just a noise word that's there for backwards compatibility with pre-role releases? 



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Murtuza Zabuawala
Date:
Subject: Re: PATCH: Login/Group Role Node
Next
From: Khushboo Vashi
Date:
Subject: Re: pgAdmin4 PATCH: Domain Module