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

SMTP Banner missing #1368

Closed
delijati opened this issue Feb 17, 2020 · 5 comments · Fixed by arother/Mailu#1 or #1553
Closed

SMTP Banner missing #1368

delijati opened this issue Feb 17, 2020 · 5 comments · Fixed by arother/Mailu#1 or #1553
Milestone

Comments

@delijati
Copy link

Hi i have two question reagrding the main.cf file.

  • How can i add the smtpd_banner = $myhostname ESMTP $mail_name (Debian/GNU)
  • When i edit the HOSTNAME in mailu.env the entry myhostname is still the old one. Can i trigger the creation of that file. [1] I tried docker container prune and this files is also not in /mailu

[1]

foo:/mailu$ docker-compose exec smtp bash -c "cat /etc/postfix/main.cf | grep myhost"
myhostname = mail.foo.de
foo:/mailu$ cat mailu.env | grep HOS
HOSTNAMES=mail.foo.de,foo-de.foo.de
@delijati
Copy link
Author

delijati commented Feb 17, 2020

Using mailu 1.7

Ok adding HOSTNAME won't work as of https://github.com/Mailu/Mailu/blob/master/core/postfix/conf/main.cf#L10

I tried overwrites:

$ cat overrides/postfix.cf 
smtpd_banner = foo-de.foo.de ESMTP Postfix (Alpine)
# and
$ docker-compose exec smtp bash -c "postconf | grep banner"
postscreen_greet_banner = $smtpd_banner
smtpd_banner = foo-de.foo.de ESMTP Postfix (Alpine)

But i still get:

$ echo QUIT|nc foo.de 25
220 mail.foo.de ESMTP ready
221 2.0.0 Bye

@delijati
Copy link
Author

Ok i also tested master but still have the same problem with not being able to set the SMTP Banner

@delijati
Copy link
Author

delijati commented Feb 18, 2020

It seams the BANNER is set by NGINX ... I changed [1] and it worked. So the overwrites in postfix.cf are complete ignored. This should go to the FAQ ;)

[1]

$ cat mailu.env | grep HOST
HOSTNAMES=foo-de.foo.de,mail.foo.de
$ sudo docker-compose stop
$ sudo docker-compose up -d
$ echo QUIT|nc foo.de 25   
220 foo-de.foo.de ESMTP ready
221 2.0.0 Bye

arother added a commit to arother/Mailu that referenced this issue Jun 26, 2020
@Nebukadneza Nebukadneza added this to the 1.8-rc milestone Aug 17, 2020
@Nebukadneza Nebukadneza added the status/response_needed Waiting for a response from the author label Aug 17, 2020
@Nebukadneza
Copy link
Member

Hi There,

The Mailu-Project is currently in a bit of a bind! We are short on man-power, and we need to judge if it is possible for us to put in some work on this issue.

To help with that, we are currently trying to find out which issues are actively keeping users from using Mailu, which issues have someone who want to work on them — and which issues may be less important. These a less important ones could be discarded for the time being, until the project is in a more stable and regular state once again.

In order for us to better assess this, it would be helpful if you could put a reaction on this post (use the 😃 icon to the top-right).

  • 👍️ if you need this to be able to use Mailu. Ideally, you’d also be able to test this on your installation, and provide feedback …
  • 🎉 if you find it a nice bonus, but no deal-breaker
  • 🚀 if you want to work on it yourself!
    We want to keep this voting open for 2 weeks from now, so please help out!

@Nebukadneza Nebukadneza removed the status/response_needed Waiting for a response from the author label Aug 23, 2020
@bors bors bot closed this as completed in ac3dfd5 Aug 23, 2020
@netbofia
Copy link

It seams the BANNER is set by NGINX ... I changed [1] and it worked. So the overwrites in postfix.cf are complete ignored. This should go to the FAQ ;)

[1]

$ cat mailu.env | grep HOST
HOSTNAMES=foo-de.foo.de,mail.foo.de
$ sudo docker-compose stop
$ sudo docker-compose up -d
$ echo QUIT|nc foo.de 25   
220 foo-de.foo.de ESMTP ready
221 2.0.0 Bye

So to clear this up I should change the order of the HOSTNAMES var to place the smtp[.example.com] alias first instead of the mail[.example.com] ou whatever was there?

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 a pull request may close this issue.

3 participants