search

albertito / chasquid

SMTP (email) server with a focus on simplicity, security, and ease of operation [mirror]
https://blitiri.com.ar/p/chasquid/
Other
868 stars 56 forks source link

smtp-check: Add flag to specify local name #38

Closed znerol closed 1 year ago

znerol commented 1 year ago

Some MTA do reject client connections unless the local name looks like an FQDN. This patch adds a command line flag to smtp-check to specify the local name to be used in the EHLO command.

Fixes #37

albertito commented 1 year ago

Hi! Thanks for sending this!

The hostname config option can be used to set the local name, and I think it does what you need. So a separate flag may not be needed (and may end up being confusing).

Is that reasonable, or am I missing something?

Note this is separate from the issue of "the default (which is not an FQDN) may cause some email rejections", let's discuss that in issue #37.

Thanks!

albertito commented 1 year ago

Sorry, I just realized this is for smtp-check which is independent from chasquid's config!

So please ignore my last message, I will review this in the next couple of days, but it seems reasonable in principle.

znerol commented 1 year ago

Merged in ad0dbb9c