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

replaced ifusio.com by ${domain} #2

Merged
merged 1 commit into from
Mar 31, 2015
Merged

Conversation

askz
Copy link
Contributor

@askz askz commented Mar 31, 2015

Your domain was in the start-mailserver.sh script, so the domain folder was never created with ENV vars.

@tomav
Copy link
Contributor

tomav commented Mar 31, 2015

Thank you @askz.

@tomav tomav closed this Mar 31, 2015
@tomav tomav reopened this Mar 31, 2015
tomav added a commit that referenced this pull request Mar 31, 2015
replaced ifusio.com by ${domain}
@tomav tomav merged commit e413dc6 into docker-mailserver:master Mar 31, 2015
@askz
Copy link
Contributor Author

askz commented Mar 31, 2015

There is still errors at the creation of the container, I'm working on it.

@askz
Copy link
Contributor Author

askz commented Mar 31, 2015

For instance, I have this :
mail_1 | Regenerating 'vmailbox' for given users mail_1 | docker_mail_users => ['max@xxx.fr|xxxxxx'] mail_1 | user 'max' for domain 'zkp.fr' with password 'xxxxxxx' mail_1 | maildirmake: /var/mail/zkp.fr/max: No such file or directory mail_1 | Postmap configurations

@askz askz deleted the patch-1 branch March 31, 2015 14:55
@askz askz restored the patch-1 branch March 31, 2015 14:55
@tomav
Copy link
Contributor

tomav commented Mar 31, 2015

I know why. At this time, we try to postmap a file that doesn't exists in the container.
It's provided by a volume containing the full /etc/postfix directory.
I'm fixing it right now.

@tomav
Copy link
Contributor

tomav commented Mar 31, 2015

@askz I just updated the repository. Please have a look to the changes.

@askz
Copy link
Contributor Author

askz commented Mar 31, 2015

Thanks, the vmail error is fixed.
but now :

mail_1 | Postfix configurations
mail_1 | sed: can't read /etc/postfix/main.cf: No such file or directory
mail_1 | postmap: fatal: open /etc/postfix/main.cf: No such file or directory

@tomav
Copy link
Contributor

tomav commented Mar 31, 2015

Hmm have you updated your local git repository? I tried locally and it works.
Or are you using the docker pre-built images tvial/docker-mailserver? (I just had a look and build has failed but it doesn't semmed to be related to the changes but a Docker infrastructure issue.)

@tomav
Copy link
Contributor

tomav commented Mar 31, 2015

The image on the registry is up to date. You can docker pull tvial/docker-mailserver.
Don't forget to remove existing container before you run the new one.

@askz
Copy link
Contributor Author

askz commented Apr 1, 2015

Okay, thank you for this, the docker-compose is now building, I can log into imap server, but smtp's refusing login.
I'm very interested in this kind of container, I'll give it a shoot when I'll have some time !

tomav added a commit that referenced this pull request Sep 12, 2015
replaced ifusio.com by ${domain}
tomav pushed a commit that referenced this pull request Apr 11, 2016
Changed $SASL_PASSWD to $SASL_PASSWORD
tve added a commit to tve/docker-mailserver that referenced this pull request May 23, 2016
tve added a commit to tve/docker-mailserver that referenced this pull request May 24, 2016
17Halbe pushed a commit to 17Halbe/docker-mailserver-1 that referenced this pull request Mar 17, 2018
17Halbe added a commit to 17Halbe/docker-mailserver-1 that referenced this pull request Apr 2, 2018
RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this pull request Aug 26, 2019
RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this pull request Aug 26, 2019
erik-wramner pushed a commit that referenced this pull request Jul 22, 2020
Small change to error message
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.

None yet

2 participants