alexyoung / ircd.js

A Node ircd (IRC daemon)
GNU General Public License v3.0
528 stars 90 forks source link

fix server password auth to require PASS as first command if config.serverPassword set #46

Closed jazzychad closed 12 years ago

jazzychad commented 12 years ago

This is a fix to require PASS to be first command from client if there is a config.serverPassword set. Previously, a client could connect w/o a password even if a config.serverPassword was set.

alexyoung commented 12 years ago

I checked the IRC RFC's PASS section and it seems to describe what your patch does, I'm not sure why I made it optionally allow USER'/NICK before PASS. I suspect I found a client working that way, but it seems like the client may have been doing it wrong.