You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2013/01/21 22:56:02 UTC

[Bug 54458] New: DataSourceRealm should provide more info on SQLException

https://issues.apache.org/bugzilla/show_bug.cgi?id=54458

            Bug ID: 54458
           Summary: DataSourceRealm should provide more info on
                    SQLException
           Product: Tomcat 7
           Version: 7.0.34
          Hardware: PC
                OS: Mac OS X 10.4
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Catalina
          Assignee: dev@tomcat.apache.org
          Reporter: mkbucc@gmail.com
    Classification: Unclassified

The getPassword() method of the DataSourceRealm does not log enough information
when it encounters an SQL error.  The catch block is:

        } catch(SQLException e) {
            containerLog.error(
                    sm.getString("dataSourceRealm.getPassword.exception",
                                 username));
        } finally {

it would be really helpful to have some details of what the SQLException is.  I
am getting this intermittently, and have resorted to subclassing getPassword()
in an attempt to figure out what is going wrong.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org

[Bug 54458] DataSourceRealm should provide more info on SQLException

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=54458

--- Comment #1 from Violeta Georgieva <vi...@apache.org> ---
Created attachment 29883
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=29883&action=edit
Patch proposal

+1

Information for SQLException can be provided for methods:
- getPassword(Connection, String)
- getRoles(Connection, String)

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org

[Bug 54458] DataSourceRealm should provide more info on SQLException

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=54458

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Mark Thomas <ma...@apache.org> ---
Thanks for the patch. It has been applied to trunk and 7.0.x and will be
included in 7.0.36 onwards.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org