Changeset 437 in code for trunk

Timestamp:

Dec 25, 2020, 12:35:20 PM (4 years ago)

Author:

hubert

Message:

Advertise all caps, CAP DEL them on registration

... so that the JOIN/history batch takes into account all capabilities.
Without this commit for example, enabling multi-prefix after the batch
makes the client send NAMES requests for all channels, which generate
needless traffic.

Location:

trunk

Files:

• downstream.go (modified) (2 diffs)
• user.go (modified) (1 diff)

trunk/downstream.go

@@ -554 +554 @@
                         if dc.capVersion, err = strconv.Atoi(args[0]); err != nil {
                                 return err
+                        }
+                }
+                if !dc.registered && dc.capVersion >= 302 {
+                        // Let downstream show everything it supports, and trim
+                        // down the available capabilities when upstreams are
+                        // known.
+                        for k, v := range needAllDownstreamCaps {
+                                dc.supportedCaps[k] = v
                         }
                 }
@@ -897 +905 @@
 
         dc.updateNick()
+        dc.updateSupportedCaps()
 
         dc.forEachUpstream(func(uc *upstreamConn) {

trunk/user.go

@@ -436 +436 @@
                                 uc.updateAway()
                         })
-
-                        dc.updateSupportedCaps()
                 case eventDownstreamDisconnected:
                         dc := e.dc