Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.
Date: 2021-02-08 04:23:05
Message-ID: CAM9w-_mOwEhf8HTXDERFBoRfB_mjJqWjDQd5r83tQ8QfUmfM7w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Rahul,

On Sat, Feb 6, 2021 at 1:25 AM Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>
wrote:

> Hi Hackers,
>
> Please find the attached patch which resolves the issue of unable to
> update column level privileges in the security tab.
>
> Additionally, when the user tries to change grantee, the sql wasn't
> generating, this too issue is resolved.
>
> This security privilege issue is resolved for the following nodes:
>
> 1.) Tables (test cases are skipped as there are no resql test cases based
> on pg & ppas)
> 2.) Columns
> 3.) Views
> 4.) MViews
> 5.) Types
> 6.) Functions
> 7.) Procedures
> 8.) Trigger functions
> 9.) Sequences
> 10.) Schema
>
> P.S for committer : Warning of whitespaces after applying patch should be
> ignored as it is a part of resql test case results.
>
Probably change the SQL generation logic to avoid the trailing whitespaces ?

>
> --
> *Rahul Shirsat*
> Senior Software Engineer | EnterpriseDB Corporation.
>

--
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | *edbpostgres.com*
<http://edbpostgres.com>
"Don't Complain about Heat, Plant a TREE"

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Yogesh Mahajan 2021-02-08 08:23:53 [pgAdmin4][Patch] - Feature Tests Failure Fix
Previous Message Sergey Burladyan 2021-02-06 04:59:58 pgagent unicode support