From b98434b5d04d1ada9b24475e17ee8947d96ad1e3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?louiz=E2=80=99?= Date: Sun, 16 Aug 2020 16:05:15 +0200 Subject: In CAP messages, handle the last arg as a list of capabilities Instead of just one. This fixes the issue of the "trailing whitespace" since we now split it on ' ' Fix #3442 --- src/irc/irc_client.cpp | 25 ++++++++++++++----------- 1 file changed, 14 insertions(+), 11 deletions(-) (limited to 'src/irc/irc_client.cpp') diff --git a/src/irc/irc_client.cpp b/src/irc/irc_client.cpp index 3ae5ac6..5f0d9b9 100644 --- a/src/irc/irc_client.cpp +++ b/src/irc/irc_client.cpp @@ -1340,19 +1340,22 @@ long int IrcClient::get_throttle_limit() const void IrcClient::on_cap(const IrcMessage &message) { const auto& sub_command = message.arguments[1]; - const auto& cap = message.arguments[2]; - auto it = this->capabilities.find(cap); - if (it == this->capabilities.end()) + const auto& caps = utils::split(message.arguments[2], ' ', false); + for (const auto& cap: caps) { - log_warning("Received a CAP message for something we didn’t ask, or that we already handled."); - return; + auto it = this->capabilities.find(cap); + if (it == this->capabilities.end()) + { + log_warning("Received a CAP message for something we didn’t ask, or that we already handled: [", cap, "]"); + return; + } + Capability& capability = it->second; + if (sub_command == "ACK") + capability.on_ack(); + else if (sub_command == "NACK") + capability.on_nack(); + this->capabilities.erase(it); } - Capability& capability = it->second; - if (sub_command == "ACK") - capability.on_ack(); - else if (sub_command == "NACK") - capability.on_nack(); - this->capabilities.erase(it); if (this->capabilities.empty()) this->cap_end(); } -- cgit v1.2.3