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