Re: [pgAdmin][RM7296]: Browser tree is not getting refresh after deleting the multiple node from collection properties

2022-04-11 Thread Akshay Joshi
Thanks, the patch applied.

On Mon, Apr 11, 2022 at 5:22 PM Pradip Parkale <
pradip.park...@enterprisedb.com> wrote:

> Hi Akshay,
>
> Please find the attached patch for the issue below.
>
>1. When deleting the constraint from the collection properties nodes,
>an error occurred
>2. The code to refresh the Dashboard after deleted session activities
>has been improved.
>
>
> On Sat, Apr 9, 2022 at 12:06 PM Akshay Joshi <
> akshay.jo...@enterprisedb.com> wrote:
>
>> Thanks, the patch applied.
>>
>> On Fri, Apr 8, 2022 at 12:29 PM Pradip Parkale <
>> pradip.park...@enterprisedb.com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Please find the attached patch for #7296.I have fixed the issue where
>>> the browser tree did not refresh after multiple nodes were deleted.
>>>
>>> --
>>> Thanks & Regards,
>>> Pradip Parkale
>>> Software Engineer | EnterpriseDB Corporation
>>>
>>
>>
>> --
>> *Thanks & Regards*
>> *Akshay Joshi*
>> *pgAdmin Hacker | Principal Software Architect*
>> *EDB Postgres *
>>
>> *Mobile: +91 976-788-8246*
>>
>
>
> --
> Thanks & Regards,
> Pradip Parkale
> Software Engineer | EnterpriseDB Corporation
>


-- 
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Principal Software Architect*
*EDB Postgres *

*Mobile: +91 976-788-8246*


Re: [pgAdmin][RM7252] 'columns to import' ... not auto selected second time for same table

2022-04-11 Thread Akshay Joshi
Thanks, the patch applied.

On Mon, Apr 11, 2022 at 4:35 PM Aditya Toshniwal <
aditya.toshni...@enterprisedb.com> wrote:

> Hi Hackers,
>
> Please review the patch to fix the issue.
>
> --
> Thanks,
> Aditya Toshniwal
> pgAdmin Hacker | Software Architect | *edbpostgres.com*
> 
> "Don't Complain about Heat, Plant a TREE"
>


-- 
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Principal Software Architect*
*EDB Postgres *

*Mobile: +91 976-788-8246*


pgAdmin 4 commit: Ensure that Columns should always be visible in the i

2022-04-11 Thread Akshay Joshi
Ensure that Columns should always be visible in the import/export dialog. Fixes 
#7252

Branch
--
master

Details
---
https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=a91762fb30edb8bb5344456754cd1da2d1b92c81
Author: Aditya Toshniwal 

Modified Files
--
docs/en_US/release_notes_6_9.rst   |  1 +
web/pgadmin/static/js/SchemaView/index.jsx | 26 -
web/pgadmin/tools/erd/static/scss/_erd.scss| 43 ++
.../javascript/SchemaView/SchemaView.spec.js   | 22 ++-
web/regression/javascript/browser/activity_spec.js | 11 +++---
.../javascript/components/CodeMirror.spec.js   | 28 ++
web/regression/javascript/fake_pgadmin.js  | 36 ++
web/regression/javascript/genericFunctions.jsx | 11 --
8 files changed, 133 insertions(+), 45 deletions(-)



Re: [pgAdmin][RM7296]: Browser tree is not getting refresh after deleting the multiple node from collection properties

2022-04-11 Thread Pradip Parkale
Hi Akshay,

Please find the attached patch for the issue below.

   1. When deleting the constraint from the collection properties nodes, an
   error occurred
   2. The code to refresh the Dashboard after deleted session activities
   has been improved.


On Sat, Apr 9, 2022 at 12:06 PM Akshay Joshi 
wrote:

> Thanks, the patch applied.
>
> On Fri, Apr 8, 2022 at 12:29 PM Pradip Parkale <
> pradip.park...@enterprisedb.com> wrote:
>
>> Hi Hackers,
>>
>> Please find the attached patch for #7296.I have fixed the issue where the
>> browser tree did not refresh after multiple nodes were deleted.
>>
>> --
>> Thanks & Regards,
>> Pradip Parkale
>> Software Engineer | EnterpriseDB Corporation
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Principal Software Architect*
> *EDB Postgres *
>
> *Mobile: +91 976-788-8246*
>


-- 
Thanks & Regards,
Pradip Parkale
Software Engineer | EnterpriseDB Corporation


refresh.patch
Description: Binary data


[pgAdmin][RM7252] 'columns to import' ... not auto selected second time for same table

2022-04-11 Thread Aditya Toshniwal
Hi Hackers,

Please review the patch to fix the issue.

-- 
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | *edbpostgres.com*

"Don't Complain about Heat, Plant a TREE"


RM7252.patch
Description: Binary data


Re: [pgAdmin4][Patch]- Feature #7012 - disable master password requirement when using alternative auth source

2022-04-11 Thread Akshay Joshi
Thanks, the patch applied.

On Mon, Apr 11, 2022 at 12:00 PM Khushboo Vashi <
khushboo.va...@enterprisedb.com> wrote:

> Hi,
>
> Please find the attached patch to implement the feature #7012 - Disable
> master password requirement when using alternative auth source
>
> When pgAdmin stores a connection password, it encrypts it using a key that
> is formed either from the master password, or from the pgAdmin login
> password for the user. In the case of auth methods such as OAuth, Kerberos
> or Webserver, pgAdmin doesn't have access to anything long-lived to form
> the encryption key from, hence it uses the master password. And if the
> master is disabled, there is no way to store the connection password.
>
> To resolve this, we have added an option to config.py (which defaults to
> None) for an alternate encryption key. pgAdmin would use this if a) the
> master password is disabled AND b) there is no suitable key/password
> available from the auth module for the user. If the option is set to
> None, pgAdmin works as it does now.
>
> Thanks,
> Khushboo
>


-- 
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Principal Software Architect*
*EDB Postgres *

*Mobile: +91 976-788-8246*


pgAdmin 4 commit: Only set permissions on the storage directory upon cr

2022-04-11 Thread Akshay Joshi
Only set permissions on the storage directory upon creation. Fixes #6958

Branch
--
master

Details
---
https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=bf974e73ce772f312736c515bcc8fcb8bd41a959

Modified Files
--
docs/en_US/release_notes_6_9.rst|  1 +
web/pgadmin/setup/data_directory.py | 15 ++-
2 files changed, 11 insertions(+), 5 deletions(-)



[pgAdmin4][Patch]- Feature #7012 - disable master password requirement when using alternative auth source

2022-04-11 Thread Khushboo Vashi
Hi,

Please find the attached patch to implement the feature #7012 - Disable
master password requirement when using alternative auth source

When pgAdmin stores a connection password, it encrypts it using a key that
is formed either from the master password, or from the pgAdmin login
password for the user. In the case of auth methods such as OAuth, Kerberos
or Webserver, pgAdmin doesn't have access to anything long-lived to form
the encryption key from, hence it uses the master password. And if the
master is disabled, there is no way to store the connection password.

To resolve this, we have added an option to config.py (which defaults to
None) for an alternate encryption key. pgAdmin would use this if a) the
master password is disabled AND b) there is no suitable key/password
available from the auth module for the user. If the option is set to None,
pgAdmin works as it does now.

Thanks,
Khushboo


RM_7012.patch
Description: Binary data