Return error for WRITE command when stream ID exceeds limit. Fixes #19

ChangeLog

@@ -1,6 +1,9 @@
 2024.338:
 	- Include server_port key in INFO CONNECTIONS response.
-	- Fix parsing of HTTP requests.
+	- Fix parsing of HTTP requests that required URL decoding.
+	- Fix and enhance HTTP /connections endpoint response.
+	- Document stream ID limitations and conventions in manual.
+	- Return error for WRITE command when stream ID exceeds limit.
 
 2024.320: v4.0.0
 	NOTE: the ring buffer format has changed in this release. Older versions

src/dlclient.c

@@ -712,6 +712,16 @@ HandleWrite (ClientInfo *cinfo)
     return -1;
   }
 
+  if (strlen (streamid) > (MAXSTREAMID - 1))
+  {
+    lprintf (1, "[%s] Error, stream ID too long: %.100s",
+             cinfo->hostname, streamid);
+
+    SendPacket (cinfo, "ERROR", "Error, stream ID too long", 0, 1, 1);
+
+    return -1;
+  }
+
   /* Set packet ID to RINGID_NONE if not provided */
   if (rv == 5 || (rv == 6 && strchr (flags, 'I') == NULL))
   {