-
Notifications
You must be signed in to change notification settings - Fork 22
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
Password warning (Mail expiration notification) #823
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
9993d9a
to
8a80bd4
Compare
8a80bd4
to
e667f78
Compare
e667f78
to
f63487e
Compare
f63487e
to
bb6f9d6
Compare
bb6f9d6
to
dc08ffe
Compare
5fda750
to
6cb55c0
Compare
6cb55c0
to
ed6f6dd
Compare
New info can be used to expose password expiration on the UI and also to send notification about password expiration
ed6f6dd
to
d4d6ec3
Compare
New info can be used to expose password expiration on the UI and also to send notification about password expiration
d4d6ec3
to
f9b679f
Compare
f9b679f
to
5ecd0b0
Compare
5ecd0b0
to
b91b317
Compare
b91b317
to
9152374
Compare
Configure mail notification for a given user domain
andre8244
reviewed
Mar 12, 2025
29a0cf8
to
57bec49
Compare
The timer will lunch the notifier script every night
The subject is now part of the notification template: this will simplify the configuration for the user.
Changes: - remove upload template action - allow only one custom template for each domain
57bec49
to
7b60438
Compare
andre8244
approved these changes
Mar 12, 2025
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 label fix and we're good, no need to request review again
5ad2620
to
3d65e74
Compare
Changes: - read custom template from new redis position - allow templating of the mail subject
If the cluster has a mail instance already installed, send the notification to the local user mail address. Other change: do not send notificaiton to user without password expiration
When the mail is sent as HTML, new lines are not preserved if the body is just plain text. Now the script tries to understand if the mail body is an HTML or not and set the mime type accordingly.
Use the user domain as for the sender address: it's better suited then the node FQDN
Suggestion from code review
Thanks to @AmaLuci
3d65e74
to
d3cf81d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Install
Install the custom core image:
Setup custom images for Samba and OpenLDAP:
Then, install:
Mail example:

Usage example (deprecated)
Using default template:
For a custom template, first create the file named custom.txt like:
Then, use:
See also: