Skip to content

update postmaster_address in dovecot config according to POSTMASTER_ADDRESS env var#866

Merged
johansmitsnl merged 2 commits intodocker-mailserver:masterfrom
andreasgerstmayr:fix-postmaster-address
Feb 27, 2018
Merged

update postmaster_address in dovecot config according to POSTMASTER_ADDRESS env var#866
johansmitsnl merged 2 commits intodocker-mailserver:masterfrom
andreasgerstmayr:fix-postmaster-address

Conversation

@andreasgerstmayr
Copy link
Copy Markdown
Contributor

Update the postmaster_address in the start-mailserver.sh script.

At the moment this config is set in the Dockerfile at: https://github.com/tomav/docker-mailserver/blob/c36e878d76827fc7062558f1edada05d7ba38880/Dockerfile#L111
But this code is executed only when the docker image is built and not at runtime.

Copy link
Copy Markdown
Contributor

@johansmitsnl johansmitsnl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you update the README file with the new variable?
When added I can merge your PR.

@andreasgerstmayr
Copy link
Copy Markdown
Contributor Author

The config variable is documented already (but didn't work correctly before): https://github.com/tomav/docker-mailserver/tree/c36e878d76827fc7062558f1edada05d7ba38880#postmaster_address

@johansmitsnl johansmitsnl merged commit 2687469 into docker-mailserver:master Feb 27, 2018
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

Successfully merging this pull request may close these issues.

2 participants