View | Details | Raw Unified | Return to bug 45453
Collapse All | Expand All

(-)apache-tomcat-5.5.26-src.orig/container/catalina/src/share/org/apache/catalina/realm/JDBCRealm.java (-2 / +3 lines)
Lines 40-46 Link Here
40
*
40
*
41
* <p><strong>TODO</strong> - Support connection pooling (including message
41
* <p><strong>TODO</strong> - Support connection pooling (including message
42
* format objects) so that <code>authenticate()</code>,
42
* format objects) so that <code>authenticate()</code>,
43
* <code>getPassword()</code> and <code>roles()</code>  do not have to be
43
* <code>getPassword()</code>, <code>roles()</code> and
44
* <code>getPrincipal()</code> do not have to be
44
* synchronized and would fix the ugly connection logic. </p>
45
* synchronized and would fix the ugly connection logic. </p>
45
*
46
*
46
* @author Craig R. McClanahan
47
* @author Craig R. McClanahan
Lines 591-597 Link Here
591
    /**
592
    /**
592
     * Return the Principal associated with the given user name.
593
     * Return the Principal associated with the given user name.
593
     */
594
     */
594
    protected Principal getPrincipal(String username) {
595
    protected synchronized Principal getPrincipal(String username) {
595
596
596
        return (new GenericPrincipal(this,
597
        return (new GenericPrincipal(this,
597
                                     username,
598
                                     username,

Return to bug 45453