Changeset 334 in code for trunk/service.go

Timestamp:

Jun 10, 2020, 2:33:13 PM (5 years ago)

Author:

fox.cpp

Message:

Do not panic if BouncerServ command without handler is sent

File:

• trunk/service.go (modified) (1 diff)

trunk/service.go

@@ -77 +77 @@
         }
 
+        if cmd.handle == nil {
+                if len(cmd.children) > 0 {
+                        var l []string
+                        appendServiceCommandSetHelp(cmd.children, words, &l)
+                        sendServicePRIVMSG(dc, "available commands: "+strings.Join(l, ", "))
+                } else {
+                        // Pretend the command does not exist if it has neither children nor handler.
+                        // This is obviously a bug but it is better to not die anyway.
+                        dc.logger.Printf("command without handler and subcommands invoked:", words[0])
+                        sendServicePRIVMSG(dc, fmt.Sprintf("command %q not found", words[0]))
+                }
+                return
+        }
+
         if err := cmd.handle(dc, params); err != nil {
                 sendServicePRIVMSG(dc, fmt.Sprintf("error: %v", err))