Re: [pgAdmin4][Patch] - RM 2186 - Support external authenticationsources [LDAP] - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgAdmin4][Patch] - RM 2186 - Support external authenticationsources [LDAP]
Date
Msg-id CA+OCxoyZvHAxNTDXYDNVE0irEFwYnJ0z-5KXKuJ_1Bc9aEYTWA@mail.gmail.com
Whole thread Raw
In response to [pgAdmin4][Patch] - RM 2186 - Support external authentication sources [LDAP]  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Responses Re: [pgAdmin4][Patch] - RM 2186 - Support external authenticationsources [LDAP]  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
List pgadmin-hackers
Hi

30 second read of the first version of the patch...

- Please move the configuration into config.py. Users should never have to modify a distributed file (it messes up packaging). I don't see any reason to use a different file just for auth config.

- I think all config options should be prefixed with LDAP_ as we may have things like CERT_FILE for other purposes too.

- I don't see any test cases.

Thanks.


On Tue, Mar 17, 2020 at 8:55 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch to support LDAP Authentication in Server mode.
To test the patch, config_auth.py needs to be configured for LDAP configurations. The config settings are explained in this file in detail. After configuring the parameters, start the pgadmin server in Server mode and connect with LDAP server with the valid user via login page.

I have tested this patch with ldap and ldap + ssl/tls. With the TLS, I have used the default config of ldap3 without certificates.

@Dave, can you please review this patch, as you have a better understanding of LDAP and you can easily pointed out if I have missed anything.

Note: For the document update I will create the task and assign to Nidhi for the same.

Thanks,
Khushboo


--
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: navnath gadakh
Date:
Subject: Re: [pgAdmin4][Patch] - RM 2186 - Support external authenticationsources [LDAP]
Next
From: Khushboo Vashi
Date:
Subject: Re: [pgAdmin4][Patch] - RM 2186 - Support external authenticationsources [LDAP]