Bug #25479
Updated by Clark ANDRIANASOLO 2 months ago
The current @60 days@ default for @rudder.users.cleanup.account.disableAfterLastLogin@ is not convenient : user that did not log in since 60 days are disabled.
The disable reason is also empty in the users table (in the statushistory column) :
<pre>
{"actor": {"name": "rudder"}, "reason": "", "actionDate": "2024-09-17T10:00:00.000Z"}, "status": "disabled"}
</pre>
We should *set set it to @never@* @never@ by default because users should be able to leave a Rudder instance unused for a long time by default and some usage already confirmed that.
We should also *add add a reason* reason in the trace, e.g. "User did not login for too long", and we should also add an appropriate *log* log (with the @application.authentication@ name, see #25478)
Also, @rudder.users.cleanup.account.deleteAfterLastLogin@ should only apply *only on already disabled users* users only (therefore the value of 120.days seems reasonable)
We should also update the doc and sample for the configuration parameters.