Function hook_data_post has 183 lines of code (exceeds 25 allowed). Consider refactoring.

Open

exports.hook_data_post = function (next, connection) {
    const cfg = this.config.get('messagesniffer.ini');
    const txn = connection?.transaction;
    if (!txn) return next();

File messagesniffer.js has 309 lines of code (exceeds 250 allowed). Consider refactoring.

Open

// messagesniffer

const fs = require('fs');
const net = require('net');
const plugin = exports;

Function hook_connect has 68 lines of code (exceeds 25 allowed). Consider refactoring.

Open

exports.hook_connect = function (next, connection) {
    const cfg = this.config.get('messagesniffer.ini');
    // Skip any private IP ranges
    // Skip connection.transaction undefined
    if (connection?.remote?.is_private || !connection?.transaction) return next();

Avoid deeply nested control flow statements.

Open

                        else if (code === 20) {
                            if (cfg.message.truncate) {
                                action = cfg.message.truncate;
                            }
                            else {

Avoid deeply nested control flow statements.

Open

                            switch (precedence) {
                                case 'bulk':
                                case 'list':
                                case 'junk':
                                    // Leave these as they are

Avoid deeply nested control flow statements.

Open

                            if (cfg.message.local_white) {
                                action = cfg.message.local_white;
                            }
                            else {
                                return next(OK);

Avoid deeply nested control flow statements.

Open

                            if (headers[headers.length-1]) {
                                headers[headers.length-1].value += `${line}\r\n`;
                            }

Avoid deeply nested control flow statements.

Open

                            if (match) {
                                headers.push({ header: match[1], value: (match[2] ? `${match[2]}\r\n` : '\r\n') });
                            }

Function hook_data_post has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

exports.hook_data_post = function (next, connection) {
    const cfg = this.config.get('messagesniffer.ini');
    const txn = connection?.transaction;
    if (!txn) return next();

• Read up
  Read up

Avoid too many return statements within this function.

Open

            return next();

Avoid too many return statements within this function.

Open

                    return next();

Avoid too many return statements within this function.

Open

                        return next(DENY, `Poor GBUdb reputation for [${connection.remote.ip}]`);

Avoid too many return statements within this function.

Open

                    return next();

Avoid too many return statements within this function.

Open

                            return next();