Don't prefix the nick with a colon while connecting to a server
authorManuel Nickschas <sputnick@quassel-irc.org>
Wed, 10 Feb 2016 18:58:52 +0000 (19:58 +0100)
committerManuel Nickschas <sputnick@quassel-irc.org>
Wed, 10 Feb 2016 19:06:02 +0000 (20:06 +0100)
For the NICK command during the initial connection handshake, we
always prefixed the parameter with a colon. While this is not an RFC
violation, it is unnecessary as nicks can't contain spaces. Worse, it
confuses some obscure ircds...

src/core/corenetwork.cpp

index dd1bf2e..a6dc785 100644 (file)
@@ -496,7 +496,7 @@ void CoreNetwork::socketInitialized()
     else {
         nick = identity->nicks()[0];
     }
-    putRawLine(serverEncode(QString("NICK :%1").arg(nick)));
+    putRawLine(serverEncode(QString("NICK %1").arg(nick)));
     putRawLine(serverEncode(QString("USER %1 8 * :%2").arg(identity->ident(), identity->realName())));
 }