Bug #25479
Updated by Clark ANDRIANASOLO 2 months ago
The current default is @60 days@ default for the @rudder.users.cleanup.account.disableAfterLastLogin@ configuration property is not convenient : any user that did not log in a period of 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 a longer period of time @never@* by default : @90 days@, because users should be able to leave a Rudder instance unused for a longer period of time, as demonstrated long time by default and some client use cases. usage already confirmed that. The configuration value should still be modifiable, and the *@never@* value should be a supported one (a documentation should be added in the @configuration.properties.sample@ file). We should also add *add a reason reason* in the trace, e.g. "User did not login for too long". long", and we should also add an appropriate *log* (with the @application.authentication@ name, see #25478) Also, @rudder.users.cleanup.account.deleteAfterLastLogin@ should only apply *only on already disabled users only users* (therefore the value of 120.days seems reasonable) We should also update the doc and sample for the configuration parameters.