Le plugin LDAP Jonmbake/discourse-ldap-auth ne fonctionne pas après la mise à niveau de discourse vers 3.0.0.beta16

Salut,

Après avoir mis à niveau Discourse vers la version 3.0.0.beta16 (7b5f7b4484), mon plugin LDAP (GitHub - jonmbake/discourse-ldap-auth: Discourse plugin to enable LDAP/Active Directory authentication.) a cessé de fonctionner avec l’erreur suivante :

« (ldap) Échec de l’authentification ! ldap_error: TypeError, aucune conversion implicite de Hash en String »

Quelqu’un rencontre-t-il le même problème ?

5 « J'aime »

Même problème ici.

J’ai essayé d’obtenir une meilleure trace pour l’erreur… mais je suis nul en ruby donc je n’ai eu aucun succès.

2 « J'aime »

Vous voudrez peut-être contacter l’auteur du plugin via les problèmes GitHub, car ce plugin n’a pas été écrit par nous chez Discourse.

2 « J'aime »

Je suis tout à fait d’accord - mais il est important de noter que le plugin n’a pas changé, par conséquent le problème a été introduit par un changement dans discourse.

2 « J'aime »

Bien sûr, mais Discourse va changer, c’est pourquoi vous recevez des notifications et appuyez sur un bouton de mise à jour. Les mises à niveau modifieront des aspects de Discourse ainsi que les versions des composants logiciels sous-jacents en permanence. Les plugins obsolètes qui supposent des versions plus anciennes de choses cesseront de fonctionner.

Je vous recommande vivement de tester les mises à niveau sur un site de staging, en particulier si vous dépendez de plugins tiers.

2 « J'aime »

Désolé - je ne montre pas du doigt si ce n’était pas clair :slight_smile:

Si l’erreur était un peu plus descriptive, je vois l’intérêt de poster sur un forum plus actif avec des développeurs qui ont récemment modifié les dépendances et la base de code. L’exposition ici est évidemment beaucoup plus grande que sur le GH d’un plugin.

Mais… un TypeError sans trace est particulièrement inutile :rire: je vais aller sur GH

4 « J'aime »

Voir TypeError with discourse 3.0.0.beta16 · Issue #71 · jonmbake/discourse-ldap-auth · GitHub. Sera mis à jour ici une fois résolu.

5 « J'aime »

Comme Tris20 l’a suggéré dans le ticket Github, mettre à jour net-ldap vers la version 0.17.1 fonctionne :partying_face:

edit : ah ! bien sûr Tris20 de Github est @Tris20 :crazy_face:, merci Tristan !

3 « J'aime »

Salut @Benjamin_D

Content que ça fonctionne :smiley:

Pour être clair, aviez-vous seulement besoin d’augmenter la version à 0.17.1 ou deviez-vous prendre le dernier commit de github ?

3 « J'aime »

est-ce le dernier commit Release 0.6.0 · jonmbake/discourse-ldap-auth@84635b1 · GitHub ?
si oui alors oui, j’ai forké la branche master et j’ai commité directement dessus (je ne suis pas encore très compétent avec github) :thinking:

3 « J'aime »

J’ai jeté un coup d’œil rapide à votre fork, il semble qu’il n’ait eu besoin que du changement 0.17.1 après tout :slight_smile:

N’hésitez pas à l’envoyer en pull request pour que @jonmbake puisse prendre la mise à jour :heart:

3 « J'aime »

Terminé, j’espère avoir trouvé les bons boutons :smile: et merci pour cette opportunité de tester le processus :hugs:

3 « J'aime »

La pull request pour la mise à niveau de net-ldap a été fusionnée et la version v0.6.1 du plugin a été publiée. :tada: Pouvez-vous vérifier que cela fonctionne, @loganharbour ou @Benjamin_D ?

6 « J'aime »

Oui - tout va bien sur mon instance de test (avec 0.6.1). Merci à tous pour votre rapidité !

3 « J'aime »