Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0038161Openbravo ERPA. Platformpublic2018-03-16 13:272019-10-20 19:33
JONHM 
alostale 
urgentmajoralways
closedfixed 
5
 
3.0PR18Q3 
caristu
Core
No
0038161: Maximum Concurrent Users limit not calculating properly
When the Maximum Concurrent Users limit is reached, the user only can log in a System User, but this session is also shown on the ad_session table with status "Concurrent Users Limit Reached" and active = Y. Those "invalid" sessions are taking into account as active users and this is causing problems with the login
3 different sessions on browsers are needed and a instance with only 2 Concurrent Global System Users:

- Log in with Openbravo in browser 1 -> successfull
- Log in with Openbravo in browser 2 -> successfull
- Log in with Openbravo in browser 3 -> concurrent user limit
- Log out in browser 2 -> The session is set as active = N
- Log in again with Openbravo in browser 2 -> concurrent user limit and only 1 valid session is active
Do not take into account the sessions with login_status = 'CUR'
No tags attached.
Issue History
2018-03-16 13:27JONHMNew Issue
2018-03-16 13:27JONHMAssigned To => platform
2018-03-16 13:27JONHMModules => Core
2018-03-16 13:27JONHMResolution time => 1522965600
2018-03-16 13:27JONHMTriggers an Emergency Pack => No
2018-04-03 12:08alostaleAssigned Toplatform => alostale
2018-04-03 12:09alostaleReview Assigned To => caristu
2018-04-03 12:14hgbotCheckin
2018-04-03 12:14hgbotNote Added: 0103641
2018-04-03 12:14hgbotStatusnew => resolved
2018-04-03 12:14hgbotResolutionopen => fixed
2018-04-03 12:14hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/9d919bcdb4fe9b9ff3960627f3c05994da53dc5c [^]
2018-04-03 12:14hgbotCheckin
2018-04-03 12:14hgbotNote Added: 0103642
2018-04-09 09:13caristuNote Added: 0103745
2018-04-09 09:13caristuStatusresolved => closed
2018-04-09 09:13caristuFixed in Version => 3.0PR18Q3
2018-05-09 19:29hudsonbotCheckin
2018-05-09 19:29hudsonbotNote Added: 0104360
2018-05-09 19:29hudsonbotCheckin
2018-05-09 19:29hudsonbotNote Added: 0104361
2019-10-20 19:33eugeniIssue Monitored: eugeni

Notes
(0103641)
hgbot   
2018-04-03 12:14   
Repository: erp/devel/pi
Changeset: 9d919bcdb4fe9b9ff3960627f3c05994da53dc5c
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Apr 03 12:06:50 2018 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/9d919bcdb4fe9b9ff3960627f3c05994da53dc5c [^]

fixed bug 38161: CUR System Admin restricted sessions are counted for CU limit

  When number of CU is reachead and user has Sys Admin role, a CUR session is
  created limiting it to oly Sys Admin role.

  These type of sessions were counted for CU limit preventing new sessions to
  be created.

  Now these sessions are not taken into account anymore for CU limit nor removed
  if limit is reached.

---
M modules/org.openbravo.client.application/src/org/openbravo/client/application/navigationbarcomponents/UserInfoComponent.java
M src/index.jsp
M src/org/openbravo/base/secureApp/HttpSecureAppServlet.java
M src/org/openbravo/erpCommon/obps/ActivationKey.java
---
(0103642)
hgbot   
2018-04-03 12:14   
Repository: erp/devel/pi
Changeset: c0b74e09ac0c460ef95d67a9c088808e11666acf
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Apr 03 12:08:23 2018 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/c0b74e09ac0c460ef95d67a9c088808e11666acf [^]

related to bug 38161: fixed typo

---
M src/org/openbravo/erpCommon/obps/ActivationKey.java
---
(0103745)
caristu   
2018-04-09 09:13   
Code reviewed + tested OK.
(0104360)
hudsonbot   
2018-05-09 19:29   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/2be7d3efe606 [^]
Maturity status: Test
(0104361)
hudsonbot   
2018-05-09 19:29   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/2be7d3efe606 [^]
Maturity status: Test