Plugin LDAP Jonmbake/discourse-ldap-auth não funciona após atualizar discourse para 3.0.0.beta16

Olá,

Depois que atualizei o Discourse para a versão 3.0.0.beta16 (7b5f7b4484), meu plugin LDAP (GitHub - jonmbake/discourse-ldap-auth: Discourse plugin to enable LDAP/Active Directory authentication.) parou de funcionar com o seguinte erro:

"(ldap) Falha na autenticação! ldap_error: TypeError, nenhuma conversão implícita de Hash em String"

Alguém está enfrentando o mesmo problema?

5 curtidas

Mesmo problema aqui.

Tentei obter um rastreamento melhor para o erro… mas sou ruim em ruby, então não tive sucesso.

2 curtidas

Você pode querer entrar em contato com o autor do plugin via issues do GitHub, pois esse plugin não foi escrito por nós na Discourse.

2 curtidas

Concordo plenamente - mas é importante notar que o plugin não mudou, portanto o problema foi introduzido por uma mudança no discourse.

2 curtidas

Claro, mas o Discourse vai mudar, é por isso que você recebe notificações e clica em um botão de atualização. Atualizações mudarão aspectos do Discourse, bem como as versões dos componentes de software subjacentes o tempo todo. Plugins desatualizados que assumem versões mais antigas de coisas, por sua vez, pararão de funcionar.

Eu recomendo fortemente que você teste as atualizações em um site de staging, especialmente se você depende de plugins de terceiros.

2 curtidas

Desculpe - não estou aqui apontando o dedo se isso não ficou claro :slight_smile:

Se o erro fosse um pouco mais descritivo, vejo valor em postar em um fórum mais ativo, junto com desenvolvedores que alteraram dependências e a base de código recentemente. A exposição aqui é obviamente muito maior do que no GH de um plugin.

Mas… um TypeError sem um rastreamento é particularmente inútil :laughing: irei para o GH

4 curtidas

Veja TypeError with discourse 3.0.0.beta16 · Issue #71 · jonmbake/discourse-ldap-auth · GitHub. Atualizarei aqui quando resolvido.

5 curtidas

Conforme Tris20 sugeriu na issue do Github, atualizar o net-ldap para 0.17.1 funciona :partying_face:

edit: ah! claro que Tris20 do Github é @Tris20 :crazy_face:, obrigado Tristan!

3 curtidas

Olá @Benjamin_D

Fico feliz que tenha funcionado :smiley:

Só para esclarecer, você só precisou aumentar a versão para 0.17.1 ou precisou pegar o commit mais recente do github?

3 curtidas

este é o commit mais recente Release 0.6.0 · jonmbake/discourse-ldap-auth@84635b1 · GitHub?
se sim, então sim, eu fiz um fork do branch master e comitei diretamente nele (ainda não sou muito proficiente com o github) :thinking:

3 curtidas

Dei uma olhada rápida no seu fork, parece que ele só precisava da alteração 0.17.1, afinal :slight_smile:

Sinta-se à vontade para enviar isso como um pull request para que @jonmbake possa pegar a atualização :heart:

3 curtidas

Feito, espero ter encontrado os botões certos :smile: e obrigado por esta oportunidade de testar o processo :hugs:

3 curtidas

A pull request para atualizar o net-ldap foi mesclada e a v0.6.1 do plugin foi lançada. :tada: Você pode verificar se funciona, @loganharbour ou @Benjamin_D?

6 curtidas

Sim - tudo está bem na minha instância de teste (com 0.6.1). Obrigado a todos pela rapidez!

3 curtidas