iDempiere monitor shows inactive clients (maintenance mode / active sessions)

Yes, i haven't though of this possibility, but in this case the System value must override the tenant value.
Thanks !

Nicolas

@Nicolas Micoud, there is still a case (tested) if GardenWorld set the SYSTEM_IN_MAINTENANCE_MODE=N and System set SYSTEM_IN_MAINTENANCE_MODE=Y in this case GardenUser still can login.

The attached patch IDEMPIERE-2516_add.patch solves that - I think it would be required to complete your approach - WDYT?

Oops, i made a mistake when uploading the 2nd patch.
Here's the "final" patch.

Thanks,

Nicolas

#1 I mean - the variable orderBy is not used

For #1, the new method is used in AdempiereMonitor.
See MClient[] clients = MClient.getAll(ctx, "AD_Client_ID");
This line replace "MClient[] clients = MClient.getAll(ctx);"

For #2, that's the case.
The MSysConfig check "AD_Client_ID IN (0, ?)" ; it's easier (almost in SaaS) to put every tenant in maintenance with a single value.
You can try in demo.idempiere.com ; put SYSTEM_IN_MAINTENANCE_MODE = Y in System and you won't be able to log in with GardenUser

Nicolas