Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Option for specifying postfix virtual_transport (in order to work with kopano as dagent) #394

Closed
alinmear opened this issue Dec 2, 2016 · 5 comments

Comments

@alinmear
Copy link
Contributor

alinmear commented Dec 2, 2016

@tomav i already made a pull request for kopano support (#335) of this project. In order to generalize this issue i would suggest instead of only making a config step for kopano we could make this one for the virtual_transport in general. This could be useful for people that need to be more flexible regarding the configuration of the postfix lmtp client and is also much cleaner (compared to my first implementation)

to make this work we would need two more environment variables:

And also for my particular needs it would be easy without double checks (see implementation of #335, check for SMTP_ONLY = 1) to integrate this into my kopano env, e.g. virtual_transport = lmtp:address-to-kopano-instance:2003.

To get an idea for this feature maybe have a look at (for a quick overview):
http://wiki.linuxquestions.org/wiki/Postfix_virtual_transport

@alinmear alinmear changed the title Add Option for specifying postfix virtual_transport (in order for work with kopano as dagent) Add Option for specifying postfix virtual_transport (in order to work with kopano as dagent) Dec 2, 2016
@tomav
Copy link
Contributor

tomav commented Dec 17, 2016

You can't override config files for that? All this kopano stuff doesn't seem mainly used.

@alinmear
Copy link
Contributor Author

@tomav, in particular this has nothing to do with kopano itself; this would implement the possibility without providing specific config files to fire up a container with a particular lmtp client (e.g. exim, postfix, sendmail, cyrus, dovecot, kopano, etc.) or to have e.g. dovecot on a different server; for this take a look at http://wiki2.dovecot.org/LMTP.

I tried to generalize the needs of my particular kopano installation, so that the whole project benefits from it.

@tomav
Copy link
Contributor

tomav commented Dec 20, 2016

Feel free to submit a PR and document this in Wiki.

alinmear added a commit to alinmear/docker-mailserver that referenced this issue Dec 20, 2016
This makes it possible to specify a lmtp config file, by providing
POSTFIX_DAGENT.

Update - Readme with informations about docker-mailserver#394

    * Add Variable ENABLE_POSTFIX_VIRTUAL_TRANSPORT (task)
    * Add Variable POSTFIX_DAGENT (section)
@alinmear
Copy link
Contributor Author

@tomav see #409

alinmear added a commit to alinmear/docker-mailserver that referenced this issue Jan 9, 2017
This makes it possible to specify a lmtp config file, by providing
POSTFIX_DAGENT.

Update - Readme with informations about docker-mailserver#394

    * Add Variable ENABLE_POSTFIX_VIRTUAL_TRANSPORT (task)
    * Add Variable POSTFIX_DAGENT (section)

Added Unit tests for virtual transport
tomav pushed a commit that referenced this issue Jan 9, 2017
* Add #394: Postfix Virtual Transport
This makes it possible to specify a lmtp config file, by providing
POSTFIX_DAGENT.

Update - Readme with informations about #394

    * Add Variable ENABLE_POSTFIX_VIRTUAL_TRANSPORT (task)
    * Add Variable POSTFIX_DAGENT (section)

Added Unit tests for virtual transport

* Fix syntax error in test/tests.bats

* Fix Unit Test
@tomav
Copy link
Contributor

tomav commented Jan 10, 2017

Merged.

@tomav tomav closed this as completed Jan 10, 2017
RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this issue Aug 26, 2019
…er#461)

* Add docker-mailserver#394: Postfix Virtual Transport
This makes it possible to specify a lmtp config file, by providing
POSTFIX_DAGENT.

Update - Readme with informations about docker-mailserver#394

    * Add Variable ENABLE_POSTFIX_VIRTUAL_TRANSPORT (task)
    * Add Variable POSTFIX_DAGENT (section)

Added Unit tests for virtual transport

* Fix syntax error in test/tests.bats

* Fix Unit Test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants