fix(logging): set max subject length in graylog logs ZMS-173 (#22)

zone-eu · Sep 6, 2024 · c566491 · c566491
1 parent c6bfdf8
commit c566491
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -73,8 +73,15 @@ module.exports.init = async app => {
         // check with zilter
         // if incorrect do app.reject()
 
+        const SUBJECT_MAX_ALLOWED_LENGTH = 16000;
         const { userName, apiKey, serverHost, zilterUrl, logIncomingData } = app.config;
 
+        let subjectMaxLength = app.config.subjectMaxLength;
+
+        if (!subjectMaxLength || subjectMaxLength > SUBJECT_MAX_ALLOWED_LENGTH) {
+            subjectMaxLength = SUBJECT_MAX_ALLOWED_LENGTH;
+        }
+
         if (logIncomingData) {
             // log available data
             app.logger.info('Incoming data: ', envelope, messageinfo, envelope.headers.getList());
@@ -184,7 +191,8 @@ module.exports.init = async app => {
         const originhost = serverHost || (envelope.originhost || '').replace('[', '').replace(']', '');
         const transhost = (envelope.transhost || '').replace('[', '').replace(']', '') || originhost;
 
-        const subject = messageinfo.subject || 'no subject';
+        let subject = messageinfo.subject || 'no subject';
+        subject = subject.substring(0, subjectMaxLength);
         const messageIdHeaderVal = allHeadersParsed['Message-ID']?.replace('<', '').replace('>', '');
 
         // Call Zilter with required params

diff --git a/wildduck-zonemta-zilter.toml b/wildduck-zonemta-zilter.toml
@@ -3,5 +3,6 @@ enabled = "receiver"
 userName = ""
 apiKey = ""
 serverHost = ""
-zilterUrl = ""          # full zilter url with https
+zilterUrl = ""           # full zilter url with https
 logIncomingData = false
+subjectMaxLength = 16000