Excellent! Is my first contribution on an open source project.
Thanks. -- Rómulo J. Rodríguez Rojas rodriguezr...@gmail.com On Tue, Aug 25, 2020 at 3:12 AM Akshay Joshi <akshay.jo...@enterprisedb.com> wrote: > 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* >
