Unable to send emails with attachments. 100% cpu usage after trying

[darthdubu]

Describe the bug

I'm unable to send emails with say a few photo attachments. Thunderbird will hang on sending and eventually deliver a timed out error as well as Maddy being at 100% cpu usage.

Steps to reproduce

Send email with multiple photos attached. I was able to send one single photo successfully, however say 3-4 will fail.

Log files

smtp: accepted	{"msg_id":"d454e23f"}
submission: incoming message	{"msg_id":"","sender":"redacted@tofu.kim","src_host":"[192.168.1.103]","src_ip":"216.15.87.92:42660","username":"redacted@tofu.kim"}
submission: RCPT ok	{"msg_id":"08ee4d3b","rcpt":"redacted@pm.me"}
submission: accepted	{"msg_id":"08ee4d3b"}
remote: QUIT error	{"msg_id":"25310311-614599f2","reason":"write tcp 10.18.2.75:57930-\u003e185.70.41.101:25: write: broken pipe","remote_server":"mail.protonmail.ch."}
queue: delivered	{"attempt":1,"msg_id":"08ee4d3b","rcpt":"redacted@pm.me"}
submission: incoming message	{"msg_id":"","sender":"redacted@tofu.kim","src_host":"[192.168.1.103]","src_ip":"redacted:42670","username":"redacted@tofu.kim"}
submission: RCPT ok	{"msg_id":"d9fa4246","rcpt":"redacted@pm.me"}

Configuration file

## Maddy Mail Server - default configuration file (2021-03-07)
# Suitable for small-scale deployments. Uses its own format for local users DB,
# should be managed via maddyctl utility.
#
# See tutorials at https://maddy.email for guidance on typical
# configuration changes.
#
# See manual pages (also available at https://maddy.email) for reference
# documentation.

# ----------------------------------------------------------------------------
# Base variables

$(hostname) = m1.tofu.kim
$(primary_domain) = tofu.kim
$(local_domains) = $(primary_domain) 7z.pm duck.casa

#log _file path_ /var/log/maddy/maddy.log
log stderr /var/log/maddy/maddy.log

tls file /data/tls_cert.pem /data/tls_key.pem
state_dir /data

# ----------------------------------------------------------------------------
# Local storage & authentication

# pass_table provides local hashed passwords storage for authentication of
# users. It can be configured to use any "table" module, in default
# configuration a table in SQLite DB is used.
# Table can be replaced to use e.g. a file for passwords. Or pass_table module
# can be replaced altogether to use some external source of credentials (e.g.
# PAM, /etc/shadow file).
#
# If table module supports it (sql_table does) - credentials can be managed
# using 'maddyctl creds' command.

auth.pass_table local_authdb {
    table sql_table {
        driver sqlite3
        dsn credentials.db
        table_name passwords
    }
}

# imapsql module stores all indexes and metadata necessary for IMAP using a
# relational database. It is used by IMAP endpoint for mailbox access and
# also by SMTP & Submission endpoints for delivery of local messages.
#
# IMAP accounts, mailboxes and all message metadata can be inspected using
# imap-* subcommands of maddyctl utility.

storage.imapsql local_mailboxes {
    driver sqlite3
    dsn imapsql.db
}

# ----------------------------------------------------------------------------
# SMTP endpoints + message routing

hostname $(hostname)

table.chain local_rewrites {
    optional_step regexp "(.+)\+(.+)@(.+)" "$1@$3"
    optional_step file /data/aliases
}

msgpipeline local_routing {
    # Insert handling for special-purpose local domains here.
    # e.g.
    # destination lists.example.org {
    #     deliver_to lmtp tcp://127.0.0.1:8024
    # }

    destination postmaster $(local_domains) {
        modify {
            replace_rcpt &local_rewrites
        }

        deliver_to &local_mailboxes
    }

    default_destination {
        reject 550 5.1.1 "User doesn't exist"
    }

#	check {
#		rspamd {
#		tag maddy
#		io_error_action ignore
#    	error_resp_action ignore
#    	add_header_action quarantine
#    	rewrite_subj_action quarantine
#    	flags pass_all
#    	}
#  	}
}

smtp tcp://0.0.0.0:25 {
    limits {
        # Up to 20 msgs/sec across max. 10 SMTP connections.
        all rate 20 1s
        all concurrency 10
    }

    max_header_size 33M
    dmarc yes
    check {
        require_mx_record
        dkim
        spf
    }

    source $(local_domains) {
        reject 501 5.1.8 "Use Submission for outgoing SMTP"
    }
    default_source {
        destination postmaster $(local_domains) {
            deliver_to &local_routing
        }
        default_destination {
            reject 550 5.1.1 "User doesn't exist"
        }
    }
}

submission tls://0.0.0.0:465 tcp://0.0.0.0:587 {
    limits {
        # Up to 50 msgs/sec across any amount of SMTP connections.
        all rate 50 1s
    }

    auth &local_authdb

    source $(local_domains) {
        check {
            authorize_sender {
                prepare_email &local_rewrites
                user_to_email identity
            }
        }

        destination postmaster $(local_domains) {
            deliver_to &local_routing
        }
        default_destination {
            modify {
                dkim $(primary_domain) $(local_domains) default
            }
            deliver_to &remote_queue
        }
    }
    default_source {
        reject 501 5.1.8 "Non-local sender domain"
    }
}

target.remote outbound_delivery {
    limits {
        # Up to 20 msgs/sec across max. 10 SMTP connections
        # for each recipient domain.
        destination rate 20 1s
        destination concurrency 10
    }
    mx_auth {
        dane
        mtasts {
            cache fs
            fs_dir mtasts_cache/
        }
        local_policy {
            min_tls_level encrypted
            min_mx_level none
        }
    }
}

target.queue remote_queue {
    target &outbound_delivery

    autogenerated_msg_domain $(primary_domain)
    bounce {
        destination postmaster $(local_domains) {
            deliver_to &local_routing
        }
        default_destination {
            reject 550 5.0.0 "Refusing to send DSNs to non-local addresses"
        }
    }
}

# ----------------------------------------------------------------------------
# IMAP endpoints

imap tls://0.0.0.0:993 { 
#tcp://0.0.0.0:143 
    auth &local_authdb
    storage &local_mailboxes
}

Environment information

• maddy version: 0.5.0

[foxcpp]

Oh, that must be #389. If you build it from master it should work fine. Gotta make a release now...

[darthdubu]

Oh okay. I'm going to rebuild and give it a shot.

[darthdubu]

Yup it's fixed. Thanks. I will note that I noticed Maddy is using almost double the amount of memory albeit it's still super lean. went from 30mb to about 60mb