juanluisbaptiste / docker-postfix

Simple SMTP relay docker image.
GNU General Public License v3.0
336 stars 166 forks source link

feat: Add message_size_limit option #76

Closed nassoor closed 2 years ago

nassoor commented 2 years ago

Description of the change

This adds the message_size_limit option to enable changing the default value.

Motivation and Context

To enable sending larger messages

How Has This Been Tested?

Tested the change with a value of 34000000 (34MB)

Types of Changes

Checklist:

And lastly, many thanks for taking your time to help us improve this project !

itsalljustdata commented 2 years ago

Winner! I was just about to do a PR for this myself....

until this gets merged, i've modified my docker run "--health-cmd" value to be --health-cmd 'postconf message_size_limit=$MESSAGE_SIZE_LIMIT && postfix status'

juanluisbaptiste commented 2 years ago

Hi @nassoor, thanks for the PR, but it currently needs rebasing.

nassoor commented 2 years ago

Done

juanluisbaptiste commented 2 years ago

@nassoor Sorry for the late reply, thanks for the rebasing but I merged a previous PR and it needs again rebasing. Please rebase against master again and I will merge inmediately. TY

nassoor commented 2 years ago

Done 👍

juanluisbaptiste commented 2 years ago

@nassoor there is still something missing as the PR still says it cannot be rebased due to conflicts.

nassoor commented 2 years ago

GitHub does not show me anything like that, quite the opposite.

https://github.com/juanluisbaptiste/docker-postfix/compare/master...nassoor:master

nassoor commented 2 years ago

I can recreate the PR if necessary.

juanluisbaptiste commented 2 years ago

This is what appears on the PR: Screenshot_20220523_231550

nassoor commented 2 years ago

I will recreate.