El plugin LDAP Jonmbake/discourse-ldap-auth no funciona después de actualizar discourse a 3.0.0.beta16

Hola,

Después de actualizar Discourse a la versión 3.0.0.beta16 (7b5f7b4484), mi plugin LDAP (GitHub - jonmbake/discourse-ldap-auth: Discourse plugin to enable LDAP/Active Directory authentication.) dejó de funcionar con el siguiente error:

«(ldap) ¡Fallo de autenticación! ldap_error: TypeError, no hay conversión implícita de Hash a String»

¿Alguien más está experimentando el mismo problema?

5 Me gusta

Mismo problema aquí.

He intentado obtener un mejor rastreo del error… pero soy malo en Ruby, así que no he tenido éxito.

2 Me gusta

Es posible que desees contactar al autor del plugin a través de los problemas de GitHub, ya que ese plugin no fue escrito por nosotros en Discourse.

2 Me gusta

Estoy totalmente de acuerdo, pero es importante tener en cuenta que el plugin no ha cambiado, por lo tanto, el problema fue introducido por un cambio en discourse.

2 Me gusta

Claro, pero Discourse va a cambiar, por eso recibes notificaciones y pulsas un botón de actualización. Las actualizaciones cambiarán aspectos de Discourse, así como las versiones de los componentes de software subyacentes todo el tiempo. Los plugins obsoletos que asumen versiones antiguas de las cosas, a su vez, dejarán de funcionar.

Recomiendo encarecidamente que pruebes las actualizaciones en un sitio de staging, especialmente si dependes de plugins de terceros.

2 Me gusta

Lo siento, no estoy señalando con el dedo si eso no quedó claro :slight_smile:

Si el error fuera un poco más descriptivo, veo valor en publicarlo en un foro más activo junto con los desarrolladores que han cambiado las dependencias y la base de código recientemente. La exposición aquí es obviamente mucho mayor que en el GH de un plugin.

Pero… un TypeError sin rastro es particularmente inútil :laughing: iré a GH.

4 Me gusta

Ver TypeError with discourse 3.0.0.beta16 · Issue #71 · jonmbake/discourse-ldap-auth · GitHub. Se actualizará aquí cuando se resuelva.

5 Me gusta

Como Tris20 sugirió en el problema de Github, actualizar net-ldap a 0.17.1 funciona :partying_face:

edición: ¡ah! por supuesto que Tris20 de Github es @Tris20 :crazy_face:, ¡gracias Tristan!

3 Me gusta

Hola @Benjamin_D

Me alegra que funcione :smiley:

Para que quede claro, ¿solo necesitabas aumentar la versión a 0.17.1 o necesitabas tomar el commit más reciente de GitHub?

3 Me gusta

¿Es este el commit más reciente Release 0.6.0 · jonmbake/discourse-ldap-auth@84635b1 · GitHub?
Si es así, entonces sí, bifurqué la rama principal y cometí directamente en ella (todavía no soy muy hábil con GitHub) :thinking:

3 Me gusta

Eché un vistazo rápido a tu fork, parece que después de todo solo necesitaba el cambio 0.17.1 :slight_smile:

Siéntete libre de enviarlo como una pull request para que @jonmbake pueda tomar la actualización :heart:

3 Me gusta

Hecho, espero haber encontrado los botones correctos :smile: y gracias por esta oportunidad de probar el proceso :hugs:

3 Me gusta

La solicitud de extracción para actualizar net-ldap se ha fusionado y la versión v0.6.1 del plugin ha sido lanzada. :tada: ¿Puedes verificar que funciona, @loganharbour o @Benjamin_D?

6 Me gusta

Sí, todo está bien en mi instancia de prueba (con 0.6.1). ¡Gracias a todos por la rapidez!

3 Me gusta