Return-Path: Mailing-List: contact tomcat-dev-help@jakarta.apache.org; run by ezmlm Delivered-To: mailing list tomcat-dev@jakarta.apache.org Received: (qmail 4155 invoked by uid 500); 6 Feb 2001 02:11:08 -0000 Delivered-To: apmail-jakarta-tomcat-4.0-cvs@apache.org Received: (qmail 4128 invoked by uid 1122); 6 Feb 2001 02:11:08 -0000 Date: 6 Feb 2001 02:11:08 -0000 Message-ID: <20010206021108.4127.qmail@apache.org> From: nacho@apache.org To: jakarta-tomcat-4.0-cvs@apache.org Subject: cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/realm JDBCRealm.java nacho 01/02/05 18:11:08 Modified: catalina/src/share/org/apache/catalina/realm JDBCRealm.java Log: Bugzilla Bugs #307 #407 *Authentication failes using the JDBCRealm with Sybase ASE 11.9.2 for Linux BugRat Report#560 reported by: a.schweitz@2bconnected.nl (Allan Schweitz) *JDBCRealm: Call trim() on names and roles read out of database BugRat Report#707 reported by arabung@experient.com (Adam Rabung) Revision Changes Path 1.9 +3 -3 jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/realm/JDBCRealm.java Index: JDBCRealm.java =================================================================== RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/realm/JDBCRealm.java,v retrieving revision 1.8 retrieving revision 1.9 diff -u -r1.8 -r1.9 --- JDBCRealm.java 2000/12/31 16:30:35 1.8 +++ JDBCRealm.java 2001/02/06 02:11:08 1.9 @@ -396,13 +396,13 @@ boolean found = false; if (rs1.next()) { if( digest.equals("") || digest.equalsIgnoreCase("No")){ - if(credentials.equals(rs1.getString(1))) { + if(credentials.equals(rs1.getString(1).trim())) { if(debug >= 2) log(sm.getString("jdbcRealm.authenticateSuccess", username)); found = true; }else if (credentials.equals( - Digest(rs1.getString(1),digest))) { + Digest(rs1.getString(1).trim(),digest))) { if (debug >= 2) log(sm.getString("jdbcRealm.authenticateSuccess", username)); @@ -424,7 +424,7 @@ preparedRoles.setString(1, username); ResultSet rs2 = preparedRoles.executeQuery(); while (rs2.next()) { - principal.addRole(rs2.getString(1)); + principal.addRole(rs2.getString(1).trim()); } rs2.close(); return (principal);