CPDSConnectionFactory.validateObject(Object) ignores Throwable
--------------------------------------------------------------

                 Key: DBCP-322
                 URL: https://issues.apache.org/jira/browse/DBCP-322
             Project: Commons Dbcp
          Issue Type: Bug
            Reporter: Sebb


CPDSConnectionFactory.validateObject(Object) catches and ignores Throwable, 
which is a bit excessive. For example:

{code}
if (rset != null) {
    try {
        rset.close();
    } catch (Throwable t) {
        // ignore
    }
}
{code}

close() can only throw SQLException, and that is all that should be ignored. In 
particular, ThreadDeath should never be ignored.

Same applies to KeyedCPDSConnectionFactory.

Basic[Managed]Datasource also catch Throwable, but rethrow it as 
SQLNestedException.
This is a bit better, but there's still the problem with ThreadDeath.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to