added coderabbitai suggestions like:

- Add error state tracking to the SyslogPrinter class.
 - configurable buffer size.
That way, anyone can -D SYSLOG_BUFFER_SIZE=256 (or larger) in their build flags to avoid silent truncation.

Author: KrX3D

wled00/syslog.cpp

@@ -120,8 +120,27 @@ size_t SyslogPrinter::write(const uint8_t *buf, size_t size) {
 }
 
 size_t SyslogPrinter::write(const uint8_t *buf, size_t size, uint8_t severity) {
-  if (!WLED_CONNECTED || buf == nullptr || !syslogEnabled) return 0;
-  if (!resolveHostname()) return 0;
+  _lastOperationSucceeded = true;
+  if (!WLED_CONNECTED) {
+    _lastOperationSucceeded = false;
+    _lastErrorMessage = F("Network not connected");
+    return 0;
+  }
+  if (buf == nullptr) {
+    _lastOperationSucceeded = false;
+    _lastErrorMessage = F("Null buffer provided");
+    return 0;
+  }
+  if (!syslogEnabled) {
+    _lastOperationSucceeded = false;
+    _lastErrorMessage = F("Syslog is disabled");
+    return 0;
+  }
+  if (!resolveHostname()) {
+    _lastOperationSucceeded = false;
+    _lastErrorMessage = F("Failed to resolve hostname");
+    return 0;
+  }
 
   syslogUdp.beginPacket(syslogHostIP, syslogPort);
 

wled00/syslog.h

@@ -49,15 +49,21 @@ class SyslogPrinter : public Print {
     WiFiUDP syslogUdp; // needs to be here otherwise UDP messages get truncated upon destruction
     IPAddress syslogHostIP;
     bool resolveHostname();
+    bool _lastOperationSucceeded;
+    String _lastErrorMessage;
 
     // Syslog configuration
     uint8_t _facility;
     uint8_t _severity;
     uint8_t _protocol;
     String _appName;
-    bool test = true;
+
     // Buffer management
-    char _buffer[128]; // Buffer for collecting characters
+    #ifndef SYSLOG_BUFFER_SIZE
+      #define SYSLOG_BUFFER_SIZE 128
+    #endif
+    char _buffer[SYSLOG_BUFFER_SIZE]; // Buffer for collecting characters
+
     size_t _bufferIndex;
     void flushBuffer();
 
@@ -75,6 +81,10 @@ class SyslogPrinter : public Print {
 
     // Severity override for specific messages
     size_t write(const uint8_t *buf, size_t size, uint8_t severity);
+
+    // Error handling
+    bool lastOperationSucceeded() const { return _lastOperationSucceeded; }
+    String getLastErrorMessage() const { return _lastErrorMessage; }
 };
 
 // Default instance