mail/postfix: Add smtp_bind_address #665
Conversation
|
The problem with postfix is there are two ways ... receiving mails (also bound IP like in sockstat) and sending mails, tcp connection always opens on demand. Receiving mails would be handled via listen address field. We can do it like in this PR, or add another 2 fields for binding to the receiving daemon, which would be more confusing IMO. |
|
Oh .. binding to sender can only be one .. so select field is not a good option if you meant this |
|
I’m that case it’s ok as is (minus multi-select) but hidden under advanced toggle... this is prone to breakage. |
|
Like this? |
| @@ -23,6 +23,12 @@ | |||
| <default>all</default> | |||
| <Required>Y</Required> | |||
| </inet_interfaces> | |||
| <bind_address type="NetworkField"> | |||
There was a problem hiding this comment.
can we not pin this to an address family? if not the validation is leaky and may break postfix config
There was a problem hiding this comment.
You mean in temlating? if contains ':' bind6 else bind4 and only one bind field?
There was a problem hiding this comment.
maybe this is more of a question for @AdSchellevis for the NetworkField type validation :)
There was a problem hiding this comment.
The problem is .. postfix has 2 directives, one for v4, one for v6, but, every directive is only allowed to have one value. I have no idea how to correctly build this .. not sure if this makes more work than solving a problem for one or two guys :)
There was a problem hiding this comment.
that's exactly why I mention it... the validation is not correct and it will create support efforts for other people who possibly cannot read but use advanced settings ... ;)
There was a problem hiding this comment.
@fichtner How to go on with this? @AdSchellevis Is validation with mask only for TextField? I tried this for NetworkField:
v4
/^\d{1,3}.\d{1,3}.\d{1,3}.\d{1,3}$/
v6
/^([0-9a-fA-F:]){1,40}$/u
But doesn't seem to work ...
|
Merged, thanks! |
https://forum.opnsense.org/index.php?topic=8649.0