Re: LDAP auth pathc

Hi Khushboo

Can you please review it.

On Mon, Aug 24, 2020 at 6:18 AM Romulo Rodriguez <rodriguezrjrr(at)gmail(dot)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(at)gmail(dot)com
>

--
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Sr. Software Architect*
*EDB Postgres <http://edbpostgres.com>*

*Mobile: +91 976-788-8246*