Re: LDAP auth pathc

The patch looks good to me.

Thanks,
Khushboo

On Mon, Aug 24, 2020 at 11:10 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> 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*
>