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
Postfix: rename "smtps" to "submissions" #3235
Conversation
Nope, just replacing We may want to add a minor test that greps 465 in I did want to get this into v12, but forgot about it. Probably should be considered a breaking change 🤷♂️ v13 release after v12.1 maybe? |
👍🏼
I thought this was fine for v12.1.0, because I see no reason that this is a breaking change. Why would you consider this a breaking change? |
If someone depends on docker-mailserver/target/postfix/master.cf Lines 30 to 31 in adb3820
Some issues that show this could happen:
Since we provide ways to target configuration for For reference, a comment motivating this Squeeze the change into v12 if you'd like? It shouldn't cause any surprises beyond concerns mentioned above, so you can probably avoid further delaying testing / freeze. If you do add it, then it should be part of the notable changes list in the changelog. |
Good points. If there isn't a huge benefit, but possible (future) problems with stuff like fail2ban expecting smtps, I think it's not worth and we shouldn't change it. |
It's fine for introducing as a breaking change. Users can raise actual bug reports to surface the problems. Like I said, I am not sure if fail2ban monitoring Users may still have their own config / scripts or other tools that are more explicitly expecting |
I had a glance at our F2B config as well, and it is using I am fine with waiting for the next major with this. It will probably not even take too long for v13.0.0 to come out, due to other changes I have in mind. |
INFO: Marked this with |
Documentation preview for this PR is ready! 🎉 Built with commit: c24b24b |
Description
@polarathene I am unsure whether we need to change even more. Please tell me if I forgot about something crucial.
Type of change
Checklist:
docs/
)