Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036384Retail ModulesStoreServerpublic2017-07-03 11:002017-07-04 09:06
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q4 
Merge Request Status
Review Assigned Tomtaal
OBNetwork customer
Support ticket
Regression levelPackaging and release
Regression date2017-06-08
Regression introduced in releaseRR17Q2
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/5b9d05b616c97890bac563e72fcc415931f75f89 [^]
Triggers an Emergency PackNo
Summary0036384: Connection leak in ServerStateBackground is there are no servers to ping
DescriptionThis changeset [1] fixed a connection leak in ServerStateBackground, but only if there was at least one store server to be pinged from the central server. If there are none, connections are still being leaked:

[1] https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/7edfb4a12cd4eec4d8a87c3a394b4c6ebd92ceae [^]
Steps To ReproduceIn a central server with no store server registered:
- Start tomcat
- Wait until the ServerStateBackground process is invoked. Each time it is invoked a connection will be leaked:

jul 03, 2017 10:56:07 AM org.apache.tomcat.jdbc.pool.ConnectionPool suspect
ADVERTENCIA: Connection has been marked suspect, possibly abandoned PooledConnection[org.postgresql.jdbc.PgConnection@5667c6bd][33259 ms.]:java.lang.Exception
    at org.apache.tomcat.jdbc.pool.ConnectionPool.getThreadDump(ConnectionPool.java:1096)
    at org.apache.tomcat.jdbc.pool.ConnectionPool.borrowConnection(ConnectionPool.java:799)
    at org.apache.tomcat.jdbc.pool.ConnectionPool.borrowConnection(ConnectionPool.java:648)
    at org.apache.tomcat.jdbc.pool.ConnectionPool.getConnection(ConnectionPool.java:200)
    at org.apache.tomcat.jdbc.pool.DataSourceProxy.getConnection(DataSourceProxy.java:128)
    at org.openbravo.apachejdbcconnectionpool.JdbcExternalConnectionPool.getConnectionFromDS(JdbcExternalConnectionPool.java:130)
    at org.openbravo.apachejdbcconnectionpool.JdbcExternalConnectionPool.getConnection(JdbcExternalConnectionPool.java:110)
    at org.openbravo.dal.core.SessionHandler.getNewConnection(SessionHandler.java:300)
    at org.openbravo.dal.core.SessionHandler.createSession(SessionHandler.java:235)
    at org.openbravo.dal.core.SessionHandler.begin(SessionHandler.java:561)
    at org.openbravo.dal.core.SessionHandler.getSession(SessionHandler.java:183)
    at org.openbravo.dal.core.SessionHandler.find(SessionHandler.java:534)
    at org.openbravo.dal.core.SessionHandler.find(SessionHandler.java:500)
    at org.openbravo.dal.service.OBDal.get(OBDal.java:353)
    at org.openbravo.mobile.core.authenticate.MobileAuthenticationKeyUtils.getKey(MobileAuthenticationKeyUtils.java:133)
    at org.openbravo.mobile.core.authenticate.MobileAuthenticationKeyUtils.encrypt(MobileAuthenticationKeyUtils.java:88)
    at org.openbravo.mobile.core.authenticate.MobileAuthenticationKeyUtils.getEncryptedAuthenticationToken(MobileAuthenticationKeyUtils.java:77)
    at org.openbravo.mobile.core.servercontroller.MobileServerUtils.getAuthenticationQueryParams(MobileServerUtils.java:198)
    at org.openbravo.mobile.core.servercontroller.ServerStateBackground$PingRunnable.run(ServerStateBackground.java:414)
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0035378 closed mtaal The ServerStateBackground process pings store server sequentially 
depends on backport 0036385RR17Q3 closed mtaal Connection leak in ServerStateBackground is there are no servers to ping 
Attached Files? 36384.export (1,172) 2017-07-03 11:18
https://issues.openbravo.com/file_download.php?file_id=10882&type=bug
Issue History
Date ModifiedUsernameFieldChange
2017-07-03 11:00AugustoMauchNew Issue
2017-07-03 11:00AugustoMauchAssigned To => AugustoMauch
2017-07-03 11:00AugustoMauchTriggers an Emergency Pack => No
2017-07-03 11:05AugustoMauchRelationship addedrelated to 0035378
2017-07-03 11:17AugustoMauchRegression level => Packaging and release
2017-07-03 11:17AugustoMauchRegression date => 2017-06-08
2017-07-03 11:17AugustoMauchRegression introduced in release => RR17Q2
2017-07-03 11:17AugustoMauchRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/5b9d05b616c97890bac563e72fcc415931f75f89 [^]
2017-07-03 11:17AugustoMauchStatusnew => scheduled
2017-07-03 11:18AugustoMauchFile Added: 36384.export
2017-07-03 12:18hgbotCheckin
2017-07-03 12:18hgbotNote Added: 0097792
2017-07-03 12:18hgbotStatusscheduled => resolved
2017-07-03 12:18hgbotResolutionopen => fixed
2017-07-03 12:18hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/d0dc67e8454982f51484d2e970dfe43530660402 [^]
2017-07-04 09:06mtaalReview Assigned To => mtaal
2017-07-04 09:06mtaalNote Added: 0097817
2017-07-04 09:06mtaalStatusresolved => closed
2017-07-04 09:06mtaalFixed in Version => RR17Q4

Notes
(0097792)
hgbot   
2017-07-03 12:18   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: d0dc67e8454982f51484d2e970dfe43530660402
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Mon Jul 03 11:04:14 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/d0dc67e8454982f51484d2e970dfe43530660402 [^]

Fixes issue 36384: Fixes connection leak in ServerStateBackground

Commits the connection outside the for loop, to ensure the connections is commited even if there are no store server to be pinged.

---
M src/org/openbravo/mobile/core/servercontroller/ServerStateBackground.java
---
(0097817)
mtaal   
2017-07-04 09:06   
Reviewed