Fix SASL on libera.chat

martynsmith / node-irc

NodeJS IRC client library

GNU General Public License v3.0

1.32k stars 425 forks source link

Open iglosiggio opened 2 years ago

iglosiggio commented 2 years ago

While deploying an IRC bot to libera.chat (w/@lartu) we were unable to connect via SASL because the ACK was never recognized.

The reason for that rested on a check that expected a weird leftover space character.

This PR checks that a trimmed version of the third argument is equal to sasl.

This is enough for our personal purposes but i'm not sure if it is a proper fix.

ralyodio commented 1 year ago

Pretty sure this project is dead. Its' been literally a year.