diff options
| author |  Sadie Powell | 2024-07-22 21:49:58 +0100 |
|---|---|---|
| committer | Sadie Powell | 2024-07-23 21:46:05 +0100 |
| commit | 81ac8ff1b8b8fa7091a2c25359152b93d9d1a49d (patch) | |
| tree | c73a62fb08eb423d49baf2da9eb8e8889b244a25 /modules/spanningtree/uid.cpp | |
| parent | Merge branch 'insp4' into master. (diff) | |
Remove support for the v3 server protocol.
Diffstat (limited to 'modules/spanningtree/uid.cpp')
| -rw-r--r-- | modules/spanningtree/uid.cpp | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/modules/spanningtree/uid.cpp b/modules/spanningtree/uid.cpp index 8425984fb..333a63caf 100644 --- a/modules/spanningtree/uid.cpp +++ b/modules/spanningtree/uid.cpp @@ -33,15 +33,14 @@ CmdResult CommandUID::HandleServer(TreeServer* remoteserver, CommandBase::Params& params) { /** - * 0 1 2 3 4 5? 6(5) 7(6) 8(7) 9(8) 10(9) (n-1) - * UID uuid nickchanged nick host dhost [user] duser ip.string signon +modes (modepara) :real + * 0 1 2 3 4 5 6 7 8 9 10 n-1 + * UID uuid nickchanged nick host dhost user duser ip.string signon +modes (modepara) :real * * The `duser` field was introduced in the 1206 (v4) protocol. */ - size_t offset = params[9][0] == '+' ? 1 : 0; time_t nickchanged = ServerCommand::ExtractTS(params[1]); - time_t signon = ServerCommand::ExtractTS(params[7+offset]); - const std::string& modestr = params[8+offset]; + time_t signon = ServerCommand::ExtractTS(params[8]); + const std::string& modestr = params[9]; // Check if the length of the uuid is correct and confirm the sid portion of the uuid matches the sid of the server introducing the user if (params[0].length() != UIDGenerator::UUID_LENGTH || params[0].compare(0, 3, remoteserver->GetId())) @@ -62,7 +61,7 @@ CmdResult CommandUID::HandleServer(TreeServer* remoteserver, CommandBase::Params else if (collideswith) { // The user on this side is fully connected, handle the collision - bool they_change = SpanningTreeUtilities::DoCollision(collideswith, remoteserver, nickchanged, params[5], params[6+offset], params[0], "UID"); + bool they_change = SpanningTreeUtilities::DoCollision(collideswith, remoteserver, nickchanged, params[5], params[7], params[0], "UID"); if (they_change) { // The client being introduced needs to change nick to uuid, change the nick in the message before @@ -74,7 +73,7 @@ CmdResult CommandUID::HandleServer(TreeServer* remoteserver, CommandBase::Params } irc::sockets::sockaddrs sa(false); - if (!sa.from(params[6+offset])) + if (!sa.from(params[7])) throw ProtocolException("Invalid IP address or UNIX socket path"); /* For remote users, we pass the UUID they sent to the constructor. @@ -86,14 +85,14 @@ CmdResult CommandUID::HandleServer(TreeServer* remoteserver, CommandBase::Params _new->ChangeRealHost(params[3], false); _new->ChangeDisplayedHost(params[4]); _new->ChangeRealUser(params[5], false); - _new->ChangeDisplayedUser(params[5+offset]); + _new->ChangeDisplayedUser(params[6]); _new->ChangeRemoteAddress(sa); _new->ChangeRealName(params.back()); _new->connected = User::CONN_FULL; _new->signon = signon; _new->nickchanged = nickchanged; - size_t paramptr = 9 + offset; + size_t paramptr = 10; for (const auto& modechr : modestr) { // Accept more '+' chars, for now |