Project

General

Profile

bug #8171

javax.sql.DataSource.getConnection() implementation in WrappedCdmDataSource fails to cache the connection which and leaves it unclosed

Added by Andreas Kohlbecker 6 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Highest
Category:
cdmlib
Target version:
Start date:
03/08/2019
Due date:
% Done:

100%

Severity:
critical
Found in Version:
Tags:

Description

When the existing cached connection in the WrappedCdmDataSource is null a new connection will be created but is not stored in the connection field. When closeOpenConnections() is being called the created connection is not closed

This issue is strongly related to #8170 but needs to be handled much earlier in order to void connections which are not closed after being used. See #8132 for symptoms.

the same account for getConnection(String username, String password)

It would be better to fix #8170, but this might require more time.

Question: Why closeOpenConnections(). The WrappedCdmDataSource always only handles only one connection and should actually not deal with more than one connection at the same time.


Related issues

Related to Edit - bug #8132: phycobank cdm-server instance problems with db connections New 02/22/2019
Related to Edit - bug #8170: javax.sql.DataSource.getConnection() implementation in WrappedCdmDataSource does not conform to the interface Feedback 03/08/2019

Associated revisions

Revision ddaf22cf (diff)
Added by Andreas Kohlbecker 6 months ago

fix #8171 internally caching new connection in WrappedCdmDataSource to avoid connection leaks

Revision 7d340a47 (diff)
Added by Andreas Kohlbecker 6 months ago

fix #8171 internally caching new connection in WrappedCdmDataSource to avoid connection leaks

History

#1 Updated by Andreas Kohlbecker 6 months ago

  • Description updated (diff)

#2 Updated by Andreas Kohlbecker 6 months ago

  • Description updated (diff)

#3 Updated by Andreas Kohlbecker 6 months ago

  • Related to bug #8132: phycobank cdm-server instance problems with db connections added

#4 Updated by Andreas Kohlbecker 6 months ago

  • Related to bug #8170: javax.sql.DataSource.getConnection() implementation in WrappedCdmDataSource does not conform to the interface added

#5 Updated by Andreas Kohlbecker 6 months ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50

#6 Updated by Andreas Kohlbecker 4 months ago

  • Description updated (diff)

#7 Updated by Andreas Kohlbecker 4 months ago

  • Status changed from Resolved to Closed
  • % Done changed from 50 to 100

We can close this ticket as solved, since fixing this issue did not cause any side effect problems

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)