Re: LDAP auth pathc - Mailing list pgadmin-hackers

From Romulo Rodriguez
Subject Re: LDAP auth pathc
Date
Msg-id CAF42czpRav_TNLMmjjAjOLDhDB2EpxyU3KEb-sBmMqyHXAPryQ@mail.gmail.com
Whole thread Raw
In response to Re: LDAP auth pathc  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Responses Re: LDAP auth pathc  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
List pgadmin-hackers
Excellent!

Is my first contribution on an open source project.

Thanks.

--

Rómulo J. Rodríguez Rojas

rodriguezrjrr@gmail.com



On Tue, Aug 25, 2020 at 3:12 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Thanks, patch applied.

On Mon, Aug 24, 2020 at 11:43 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
The patch looks good to me. 

Thanks,
Khushboo

On Mon, Aug 24, 2020 at 11:10 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Khushboo

Can you please review it.

On Mon, Aug 24, 2020 at 6:18 AM Romulo Rodriguez <rodriguezrjrr@gmail.com> wrote:
Buenas tardes.

Recientemente configuré PgAdmin 4.25 con autenticación LDAP y pasé varias horas tratando de encontrar el error ya que siempre obtenía como resultado "Could not find the specified user.".

Me percate que el código del método LDAPAuthentication.search_ldap_user() no funciona sobre un un directorio con una gran cantidad de resultados. Por defecto los servidores como OpenLDAP limitan la cantidad de resultados que puede ser obtenido por un cliente, normalmente solo las cuentas administradoras pueden iterar sobre todo el directorio.

El código en la función indicada se basa en iterar todos los resultados obtenidos y comparar el nombre de usuario, lo cual es ineficiente y provoca que en directos muy extensos los usuarios no sean encontrados.

Adjunto un patch del archivo ldap.py que simplifica la búsqueda agregando a la condición de búsqueda del usuario directamente en el filtro aplicada en la búsqueda.

Quedo atento a cualquier observación.

Saludos y muy agradecido por su trabajo.

--

Rómulo J. Rodríguez Rojas

rodriguezrjrr@gmail.com



--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Sr. Software Architect
EDB Postgres
Mobile: +91 976-788-8246



--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Sr. Software Architect
EDB Postgres
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website
Next
From: Ashesh Vashi
Date:
Subject: Re: LDAP auth pathc