It is not clear to me what value to enter in the 'Password Expiration Warning' field.

The description says: 'The comma separated list of days. The warning about expiration of the password will be sent out on those days before the expiration. Leaving this field empty won't send out or display any warnings.'

Maybe also add an example value to the field description.

Is a correct value 'Monday,Tuesday' ? Or do you need to enter weekday numbers, like '1,2' Or are they the day numbers in a month ? Like every 10th and 20th of a month, like '10,20' ? Or is it X days before expiration, so if your passwords are valid for 60 days and you want to sent a warning after 50 days, you would have to enter '10' ?

So just a little bit more info would clear up confusion I think.

I worked out it is the last option: X number of days before the expiration date by studying the password_policy_expiration.test file.

I would change this text:
'The comma separated list of days. The warning about expiration of the password will be sent out on those days before the expiration. Leaving this field empty won't send out or display any warnings.'

To this:
'The comma separated list of days, describing the number of how many days before the expiration date the warning mails about password expiration should be send, for example '14,7' meaning: send a warning mail two weeks and one week before password expiration date. Leaving this field empty won't send out or display any warnings.'

Comments

flyke created an issue. See original summary.

aohrvetpv’s picture

StatusFileSize
new14.4 KB

Agreed that it is unclear. We can clarify the description. Using a "days before password expiration" as a field suffix seems like it may simplify the clarified description. See screenshot.

aohrvetpv’s picture

Version: 7.x-1.12 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new1.75 KB

Patch matching screenshot in #2

  • AohRveTPV committed ec135a1 on 7.x-1.x
    Issue #2627908 by AohRveTPV: Password Expiration Warning: not entirely...
aohrvetpv’s picture

Committed as in #3, except changed "multiple numbers" to "numbers". ("multiple numbers" was redundant because plurality is indicated by "numbers" alone.)

aohrvetpv’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)
aohrvetpv’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new1010 bytes

  • AohRveTPV committed a2c8bd7 on 6.x-1.x
    Issue #2627908 by AohRveTPV: Password Expiration Warning: not entirely...
aohrvetpv’s picture

Status: Needs review » Fixed

8.x-3.x does not yet have expiration warning e-mails.

The warning in 7.x-2.x seems clear enough to me:
'A comma separated list of time intervals, when to send warning e-mails. (Use normal English, like 90 days or 5 hours.) If prefixed with a negative (like -2 days) then this will be before the expiration. If left empty, no e-mail will be sent.'

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.