The patch looks good to me. Thanks, Khushboo
On Mon, Aug 24, 2020 at 11:10 AM Akshay Joshi <akshay.jo...@enterprisedb.com> wrote: > Hi Khushboo > > Can you please review it. > > On Mon, Aug 24, 2020 at 6:18 AM Romulo Rodriguez <rodriguezr...@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 >> >> rodriguezr...@gmail.com >> > > > -- > *Thanks & Regards* > *Akshay Joshi* > *pgAdmin Hacker | Sr. Software Architect* > *EDB Postgres <http://edbpostgres.com>* > > *Mobile: +91 976-788-8246* >
