-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fix: removed deprecated smtp/ldap options #1912
Conversation
This removes the deprecated options from 4.25. This includes the LDAP filters which allow {0} or {1} placeholders. The new aliases are documented. Additionally it refactors the keys validator to use uniform messages for most replaced keys.
ArtifactsThese changes are published for testing on Buildkite, DockerHub and GitHub Container Registry. Docker Container
|
Codecov Report
@@ Coverage Diff @@
## master #1912 +/- ##
==========================================
- Coverage 72.08% 71.86% -0.22%
==========================================
Files 152 152
Lines 4345 4319 -26
Branches 179 179
==========================================
- Hits 3132 3104 -28
- Misses 991 995 +4
+ Partials 222 220 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple small changes but otherwise LGTM.
c8fbc42
to
dfee0bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
This removes the deprecated options from 4.25. This includes the LDAP filters which allow {0} or {1} placeholders. The new aliases are documented. Additionally it refactors the keys validator to use uniform messages for most replaced keys.
Fix because they should have been removed in 4.28.0, but will be removed in 4.28.1 instead. Also it's noteworthy that the coverage has decreased, it appears it's because there were a lot of previously covered lines completely removed mostly. I couldn't see any other negative hits though if you see one let me know.