Re: LDAP auth pathc

Excellent!

Is my first contribution on an open source project.

Thanks.

--

Rómulo J. Rodríguez Rojas

rodriguezrjrr(at)gmail(dot)com

On Tue, Aug 25, 2020 at 3:12 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Thanks, patch applied.
>
> On Mon, Aug 24, 2020 at 11:43 AM Khushboo Vashi <
> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>
>> 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*
>>>
>>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Sr. Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>