PERMIT_DOCKER=none as new default value#2424
PERMIT_DOCKER=none as new default value#2424georglauterbach merged 10 commits intodocker-mailserver:masterfrom
Conversation
Wouldn't that qualify for a breaking change? |
|
My opinion: definitely. But see our maintainer discussion from 19. Dec and the provided example, which also dropped functionality. tl;dr. if the container comes up without interaction, it's not a breaking change and people should read the changelog + adjust their config. |
|
Breaking change should probably wait until next major? What's the impact of existing users that upgrade and don't adjust the ENV accordingly? How soon would they realize something like this happened? (assuming they don't read changelog and expect things not to break from a minor release) |
Two things I can think of:
|
Sounds like it warrants waiting on major version then? I know we're a little less strict on that if the change isn't likely to affect many users in practice and our release notes clearly cover the change. I'm fine with this being a minor version change 👍 |
polarathene
left a comment
There was a problem hiding this comment.
LGTM, may want to adjust some log messages though?
Co-authored-by: Brennan Kinney <[email protected]>
|
I have an upcoming PR as well that changes some default values (unifies ENV setup). These changes would technically be identical in impact to the changes in this PR. IMO we should wait for #2419 ad #2420 to be merged, release |
|
If this PR can go into 10.6.0, why not straight into 10.5.0? Whats the benefit in waiting? |
Fair point, and very much fine with me :) |
|
I still See it as a breaking Change and therefore a Major Version bump. If we wajt to avoid a Lot of Major Version bumps we could:
Ideally we should also have a grace period to warn Users already 1-2 Version ahead about upcoming breaking Changes behaviors. |
While this would be optimal, we're maintaining this in our free time and we do not have the resources to plan one or two (even minor) versions ahead. I've got no problem with releasing |
|
We could also have a separate PR that is long-lived open which is a We'd just have the long-lived PR frequently syncing up with |
This sounds like a possible solution, but we'd need to make sure no |
|
It would also need some extra care with merging as we'd want to rebase the commits instead of squash merge for such a branch AFAIK when merging to master for a release. Might also be important to make that branch protected as well. |
|
Or Just Release New Major Versions If a breaking Change occurs 😅 |
Does make our docs version selector less pleasant to use though 😅 The separate branch approach is only useful for queuing up changes for a major release, which isn't that common of a need. Maybe you've got a good point though, and major version releases don't need to be few and far between, but treated just like minor/patch releases.. |
|
So, according to the maintainers' discussion, this should be tagged |
|
@casperklein we can now go ahead and start merging the open PRs. I'd start with the numerically lowest, i.e. this one and work our way forward. |
I good plan. However, I just updated the branch in #2434 to merge next 😆 Let's merge this and then from low to high. |
|
Documentation preview for this PR is ready! 🎉 Built with commit: 5053132 |
Description
Using DMS with podman can lead to security issues (e.g. an open relay), if not done properly. This PR aims a more secure out-of-the-box setup for podman users.
The following changes have been done:
none.none.PERMIT_DOCKER=containermust be used.Related to: #2393
Type of change
Checklist:
docs/)