Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation for this change
No security updates that I can see in change log: https://www.opensmtpd.org/announces/release-6.0.3.txt.
This release introduces 'configurable sub-address delimiters' (which we had on Nix via a compile-time argument that has now been deprecated).
There's also a mention in the changelog: 'unplug filter code from the smtp engine'. Indeed, the patch that we use to specific
PATH_LIBEXEC
at runtime had a part for taking care of filters and that part seem to no longer correspond to anything in the code, so I've removed it from the patch.I've also freshened up the rest of the patch file.
It'd be great if someone who runs an opensmtpd service could test this.
cc @joachifm @romildo @dtzWill @rickynils
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)