}
}
+/* ERROR - "ERROR :reason"
+Example: ERROR :Closing Link: nickname[xxx.xxx.xxx.xxx] (Large base64 image paste.)
+See https://tools.ietf.org/html/rfc2812#section-3.7.4 */
+void CoreSessionEventProcessor::processIrcEventError(IrcEvent *e)
+{
+ if (!checkParamCount(e, 1))
+ return;
+
+ if (coreNetwork(e)->disconnectExpected()) {
+ // During QUIT, the server should send an error (often, but not always, "Closing Link"). As
+ // we're expecting it, don't show this to the user.
+ e->setFlag(EventManager::Silent);
+ }
+}
+
#ifdef HAVE_QCA2
void CoreSessionEventProcessor::processKeyEvent(KeyEvent *e)
return;
QString channel = e->params()[0];
- IrcUser *ircuser = e->network()->ircUser(e->params()[4]);
+ // Store the nick separate from ircuser for AutoWho check below
+ QString nick = e->params()[4];
+ IrcUser *ircuser = e->network()->ircUser(nick);
if (ircuser) {
+ // Only process the WHO information if an IRC user exists. Don't create an IRC user here;
+ // there's no way to track when the user quits, which would leave a phantom IrcUser lying
+ // around.
+ // NOTE: Whenever MONITOR support is introduced, the IrcUser will be created by an
+ // RPL_MONONLINE numeric before any WHO commands are run.
processWhoInformation(e->network(), channel, ircuser, e->params()[3], e->params()[1],
e->params()[2], e->params()[5], e->params().last().section(" ", 1));
}
// Check if channel name has a who in progress.
- // If not, then check if user nick exists and has a who in progress.
+ // If not, then check if user nickname has a who in progress. Use nick directly; don't use
+ // ircuser as that may be deleted (e.g. nick joins channel, leaves before WHO reply received).
if (coreNetwork(e)->isAutoWhoInProgress(channel) ||
- (ircuser && coreNetwork(e)->isAutoWhoInProgress(ircuser->nick()))) {
+ (coreNetwork(e)->isAutoWhoInProgress(nick))) {
e->setFlag(EventManager::Silent);
}
}
return;
QString channel = e->params()[1];
- IrcUser *ircuser = e->network()->ircUser(e->params()[5]);
+ QString nick = e->params()[5];
+ IrcUser *ircuser = e->network()->ircUser(nick);
if (ircuser) {
+ // Only process the WHO information if an IRC user exists. Don't create an IRC user here;
+ // there's no way to track when the user quits, which would leave a phantom IrcUser lying
+ // around.
+ // NOTE: Whenever MONITOR support is introduced, the IrcUser will be created by an
+ // RPL_MONONLINE numeric before any WHO commands are run.
processWhoInformation(e->network(), channel, ircuser, e->params()[4], e->params()[2],
e->params()[3], e->params()[6], e->params().last());
// Don't use .section(" ", 1) with WHOX replies, for there's no hopcount to trim out
}
// Check if channel name has a who in progress.
- // If not, then check if user nick exists and has a who in progress.
+ // If not, then check if user nickname has a who in progress. Use nick directly; don't use
+ // ircuser as that may be deleted (e.g. nick joins channel, leaves before WHO reply received).
if (coreNetwork(e)->isAutoWhoInProgress(channel) ||
- (ircuser && coreNetwork(e)->isAutoWhoInProgress(ircuser->nick()))) {
+ (coreNetwork(e)->isAutoWhoInProgress(nick))) {
e->setFlag(EventManager::Silent);
}
}