From b6935108cac7395565034db58d21b7a861133083 Mon Sep 17 00:00:00 2001 From: Peter Powell Date: Fri, 19 Apr 2019 14:31:43 +0100 Subject: Fall back to the most recent version supported on "CAP LS". This ensures compliance with the behaviour specified in ircv3/ircv3-specifications#371. --- src/modules/m_cap.cpp | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/modules/m_cap.cpp b/src/modules/m_cap.cpp index 922061757..6e3033bf2 100644 --- a/src/modules/m_cap.cpp +++ b/src/modules/m_cap.cpp @@ -407,13 +407,21 @@ class CommandCap : public SplitCommand } else if ((subcommand == "LS") || (subcommand == "LIST")) { + Cap::Protocol capversion = Cap::CAP_LEGACY; const bool is_ls = (subcommand.length() == 2); - if ((is_ls) && (parameters.size() > 1) && (parameters[1] == "302")) - manager.Set302Protocol(user); + if ((is_ls) && (parameters.size() > 1)) + { + unsigned int version = ConvToNum(parameters[1]); + if (version >= 302) + { + capversion = Cap::CAP_302; + manager.Set302Protocol(user); + } + } std::string result; // Show values only if supports v3.2 and doing LS - manager.HandleList(result, user, is_ls, ((is_ls) && (manager.GetProtocol(user) != Cap::CAP_LEGACY))); + manager.HandleList(result, user, is_ls, ((is_ls) && (capversion != Cap::CAP_LEGACY))); DisplayResult(user, subcommand, result); } else if ((subcommand == "CLEAR") && (manager.GetProtocol(user) == Cap::CAP_LEGACY)) -- cgit v1.2.3