Re: [pgAdmin][RM5516]: Added support of Row Security Policies - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: [pgAdmin][RM5516]: Added support of Row Security Policies
Date
Msg-id CANxoLDfBYPQ63AfCD=vbhwz4GLmY4a_pdOMa0YcUa1BU7bT5qQ@mail.gmail.com
Whole thread Raw
In response to [pgAdmin][RM5516]: Added support of Row Security Policies  (Pradip Parkale <pradip.parkale@enterprisedb.com>)
Responses Re: [pgAdmin][RM5516]: Added support of Row Security Policies  (Pradip Parkale <pradip.parkale@enterprisedb.com>)
List pgadmin-hackers
Thanks, patch applied with the following modifications:

  • js file name should be "row_security_policy.js" instead of "row_security_policies.js".
  • Node name should be "row_security_policy" instead of "row_level_policies". Node name should not be pluralized and it matches the folder name.
  • Icon name should be "coll-row_security_policy.svg" and "row_security_policy.svg", it should matches with the Node name.
  • The 'Owner' should be renamed to 'Role' and there must be a role 'public' in the drop-down list. If the user does not provide any role then by default it is public.
  • Keywords like select, where, As should be in upper case in Jinja template.
  • The policy node is disappeared when updating any value. Fixed that issue.
  • There is no icon showing after creating the RLS node.


On Fri, Jun 12, 2020 at 4:45 PM Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached patch for the 'Row Level Security' feature.
I have added RE-SQL and API test cases also.

To run the RE-SQL cases - 
  • python runtests.py --pkg resql --modules row_security_policies.
To run the API cases -
  •  python runtests.py --pkg browser.server_groups.servers.databases.schemas.tables.row_security_policies.tests

--
Thanks & Regards,
Pradip Parkale
QMG, EnterpriseDB Corporation


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Added support of Row Security Policies. Fixes #5516
Next
From: Khushboo Vashi
Date:
Subject: Re: [pgAdmin][Patch] RM4226 - Table Properties - select all checkboxonly selects the first 50 tables