search

sbp / saxo

Quick and flexible irc bot, extensible in any language
http://inamidst.com/saxo/
Apache License 2.0
52 stars 22 forks source link

Make the connection step more IRCd cross-compatible #41

Closed musalbas closed 10 years ago

musalbas commented 10 years ago

Some popular IRCds, such as charybdis, will send you a 451 "not registered" error if you send a JOIN or WHO command before responding to the initial PING request the server sends you, hence saxo fails to connect. This change makes it so that saxo only sends those commands after receiving the RPL_WELCOME message (001), as specified in RFC 2812.

Some popular IRCds, such as charybdis, also only accept PONG replies that match the PING argument sent by the server. This commit makes it so that saxo respects that.

sbp commented 10 years ago

Thanks!

sbp commented 10 years ago

Thanks also to @dpk, for #2!