Skip to content
GitLab
Closed
Issue created by Côme Chilliet@cchillietReporter

Create an expiration alert mail script

Closed

Create an expiration alert mail script

Create a script made to run as a cron, checking which users are soon to be expired and sending them alert mail. It must store tokens and send a link to withdraw the expiration. Create a PHP script which handles the clic on the links.

Optionally the manager of the user, or of the OU the user is in could be notified of such operations.

(from redmine: issue id 4775, created on 2016-05-11, closed on 2016-06-10)

  • Relations:
    • relates #4785
    • relates #4864
    • copied_to #4809
  • Changesets:
    • Revision 40efab17 by Côme Chilliet on 2016-05-19T09:55:24.000Z:
Fixes #4775 Added first draft of user-reminder plugin
  • Revision 74af9a54 by Côme Chilliet on 2016-05-19T10:19:37.000Z:
Fixes #4775 Added missing comma
  • Revision 67e33235 by Côme Chilliet on 2016-05-19T11:11:49.000Z:
Fixes #4775 fixed a few more things
  • Revision 9c1c4614 by Côme Chilliet on 2016-05-25T09:28:22.000Z:
Fixes #4775 Added first draft of user-reminder plugin
  • Revision 78eaec8a by Côme Chilliet on 2016-05-25T09:28:32.000Z:
Fixes #4775 Added missing comma
  • Revision 711a6494 by Côme Chilliet on 2016-05-25T09:28:40.000Z:
Fixes #4775 fixed a few more things
  • Revision 138ef425 by Benoit MORTIER on 2016-05-31T15:25:11.000Z:
Fixes: #4775 Create an expiration alert mail script

Adding the config file

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 9ee4d84d by Côme Chilliet on 2016-06-02T05:08:55.000Z:
Fixes #4775 Added an error message when configfile is missing, and trying to avoid using manager_mail if it’s not defined
  • Revision adf943df by Côme Chilliet on 2016-06-02T05:10:29.000Z:
Fixes #4775 Added an error message when configfile is missing, and trying to avoid using manager_mail if it’s not defined
  • Revision f483b46b by Benoit MORTIER on 2016-06-03T10:01:03.000Z:
Fixes: #4775 Create an expiration alert mail script

Adding the config file

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 0dc03c78 by Côme Chilliet on 2016-06-08T08:58:06.000Z:
Fixes #4775 Added email content configuration to user-reminder
  • Revision 70ecc419 by Côme Chilliet on 2016-06-08T09:00:24.000Z:
Fixes #4775 Added email content configuration to user-reminder
  • Revision cead4810 by Côme Chilliet on 2016-06-08T10:00:32.000Z:
Fixes #4775 forwarding mail to manager
  • Revision 72424d3e by Côme Chilliet on 2016-06-08T10:01:12.000Z:
Fixes #4775 forwarding mail to manager
  • Revision 1a147588 by Côme Chilliet on 2016-06-09T10:32:31.000Z:
Fixes #4775 Fixed problems in user-reminder, switch to its own tpl
  • Revision 0459943f by Benoit MORTIER on 2016-06-09T14:50:21.000Z:
Fixes: #4775 Create an expiration alert mail script

add missing global config

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 49dbe7a9 by Côme Chilliet on 2016-06-09T15:12:40.000Z:
Fixes #4775 Fixed problems in user-reminder, switch to its own tpl

Conflicts:
	user-reminder/html/class_expiredUserPostpone.inc
  • Revision 5ae21dd1 by Côme Chilliet on 2016-06-09T15:16:35.000Z:
Fixes #4775 Removed useless part of tpl, fixed stupid error message when mail fails
  • Revision 6d074760 by Côme Chilliet on 2016-06-09T15:18:20.000Z:
Fixes #4775 Removed useless part of tpl, fixed stupid error message when mail fails
  • Revision e3ccc4a0 by Côme Chilliet on 2016-06-09T15:21:59.000Z:
Fixes #4775 fixed an error about variables names
  • Revision 6089a977 by Côme Chilliet on 2016-06-09T15:22:35.000Z:
Fixes #4775 fixed an error about variables names
  • Revision 9aa89905 by Côme Chilliet on 2016-06-10T06:48:55.000Z:
Fixes #4775 avoid PHP error when user is not found in the LDAP
  • Revision aabafde7 by Côme Chilliet on 2016-06-10T06:50:03.000Z:
Fixes #4775 avoid PHP error when user is not found in the LDAP
  • Revision 0670de10 by Côme Chilliet on 2016-06-10T08:00:59.000Z:
Fixes #4775 added back needed javascript call
  • Revision 820b8727 by Côme Chilliet on 2016-06-10T08:27:21.000Z:
Fixes #4775 added back needed javascript call
  • Revision 2061cdd8 by Côme Chilliet on 2016-06-10T09:04:20.000Z:
Fixes #4775 Fixed reading of from email address from configuration is expiration postpone
  • Revision 8420f580 by Côme Chilliet on 2016-06-10T09:05:08.000Z:
Fixes #4775 Fixed reading of from email address from configuration is expiration postpone
  • Revision 74c590d0 by Côme Chilliet on 2016-06-10T09:32:14.000Z:
Fixes #4775 deleting tokens after use
  • Revision 19e3d389 by Côme Chilliet on 2016-06-10T09:33:09.000Z:
Fixes #4775 deleting tokens after use
  • Revision 85e8b85b by Côme Chilliet on 2016-06-10T09:46:49.000Z:
Fixes #4775 Avoiding error message when token is not existing
  • Revision cb81cada by Côme Chilliet on 2016-06-10T09:47:24.000Z:
Fixes #4775 Avoiding error message when token is not existing
  • Revision 9b3c2749 by Côme Chilliet on 2016-06-10T09:53:12.000Z:
Fixes #4775 avoiding double error message
  • Revision 06a1efd7 by Côme Chilliet on 2016-06-10T09:53:47.000Z:
Fixes #4775 avoiding double error message
  • Revision 211649b6 by Benoit MORTIER on 2016-06-10T18:02:24.000Z:
Fixes #4775 remove ssl warning

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 140a1f37 by Benoit MORTIER on 2016-06-10T21:17:32.000Z:
Fixes #4775 remove form with all to recovery.php, remove focus_field

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 9c60696f by Benoit MORTIER on 2016-06-10T21:18:25.000Z:
Fixes #4775 remove form with all to recovery.php, remove focus_field

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 2b99c3d4 by Benoit MORTIER on 2016-06-10T21:32:54.000Z:
Fixes #4775 script is now part of argonaut

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 7cf6e30f by Benoit MORTIER on 2016-06-10T21:33:29.000Z:
Fixes #4775 script is now part of argonaut

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 79ca335a by Benoit MORTIER on 2016-06-19T21:03:05.000Z:
Fixes #4775 remove ssl warning

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Custom Fields:
    • Bug in version: 1.0.12
  • Uploads:
    • Screenshot_at_17-00-51
    • 1465492389.5

Tasks

0

Activity

  • bmortier
    bmortier @bmortier ·
    Maintainer

    hello,

    we use libemail-mime-perl et libemail-sender-perl for that

    Cheers

    (from redmine: written on 2016-05-19)

  • bmortier
    bmortier @bmortier ·
    Maintainer

    hello,

    the script exit with

    perl fusiondirectory-user-reminder --list-vars Can't call method "val" on an undefined value at fusiondirectory-user-reminder line 218.

    on all case if the configuration file is not correct, it should do like fusiondirectory-setup if no argument is done or configuration file is not right

    Cheers

    (from redmine: written on 2016-05-31)

  • Jonathan Swaelens
    Jonathan Swaelens @jswaelens ·
    Developer

    If we don't add a manager we have the next message.

    Use of uninitialized value $manager_cn in concatenation (.) or string at ./fd-reminder line 395.
Use of uninitialized value $manager_mail in concatenation (.) or string at ./fd-reminder line 395.

    (from redmine: written on 2016-05-31)

  • bmortier
    bmortier @bmortier ·
    Maintainer

    hello,

    the command miss a manpage

    Cheers

    (from redmine: written on 2016-06-03)

  • Jonathan Swaelens
    Jonathan Swaelens @jswaelens ·
    Developer

    It's ok I don't have uninitialized errors anymore.

    (from redmine: written on 2016-06-03)

  • bmortier
    bmortier @bmortier ·
    Maintainer

    Hello,

    finally this script will only need small adjustment to get more variables from the ldap.

    base_url subject (subject of the mail) body (body of the mail including vars like the recovery email body)

    Another linked ticket will be made for the conversion to the argonaut framework

    it will be moved to argonaut source before FD 1.0.13 will be released

    Cheers

    (from redmine: written on 2016-06-03)

  • Jonathan Swaelens
    Jonathan Swaelens @jswaelens ·
    Developer

    Hello,

    When I click on the link with the token I have the next error

    Fatal error: Call to a member function get_ldap_link() on null in /usr/share/fusiondirectory/html/class_expiredUserPostpone.inc on line 134

    It do the same if my user have a manager or not.

    Cheers

    (from redmine: written on 2016-06-09)

  • bmortier
    bmortier @bmortier ·
    Maintainer

    hello,

    fixed in 1.0.12-fixes, it was missing global $config;

    Cheers

    (from redmine: written on 2016-06-09)

  • Jonathan Swaelens
    Jonathan Swaelens @jswaelens ·
    Developer

    Hello,

    When I use the link to postpone an user I have the error that you see in the screenshot.

    My user don't have any manager and his email is true@voila.fr

    Cheers.

    (from redmine: written on 2016-06-09)

  • Jonathan Swaelens
    Jonathan Swaelens @jswaelens ·
    Developer

    Hello,

    With the last fix I have now the next error when I click on the postpone link (look at the screenshot).

    Cheers.

    (from redmine: written on 2016-06-09)

  • Jonathan Swaelens
    Jonathan Swaelens @jswaelens ·
    Developer

    Close issue

    (from redmine: written on 2016-06-10)

  • Jonathan Swaelens closed

    closed

  • bmortier added Added label

    added Added label

Please register or sign in to reply