You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by fs...@apache.org on 2016/03/09 20:48:17 UTC
svn commit: r1734301 -
/tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java
Author: fschumacher
Date: Wed Mar 9 19:48:17 2016
New Revision: 1734301
URL: http://svn.apache.org/viewvc?rev=1734301&view=rev
Log:
Use multicatch to reduce code duplication.
Modified:
tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java
Modified: tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java?rev=1734301&r1=1734300&r2=1734301&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java (original)
+++ tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java Wed Mar 9 19:48:17 2016
@@ -1266,8 +1266,9 @@ public class JNDIRealm extends RealmBase
// Authenticate the specified username if possible
principal = authenticate(context, username, credentials);
- } catch (NullPointerException e) {
- /* BZ 42449 - Kludge Sun's LDAP provider
+ } catch (NullPointerException | CommunicationException
+ | ServiceUnavailableException e) {
+ /* BZ 42449 - Catch NPE - Kludge Sun's LDAP provider
with broken SSL
*/
// log the exception so we know it's there.
@@ -1282,37 +1283,6 @@ public class JNDIRealm extends RealmBase
// Try the authentication again.
principal = authenticate(context, username, credentials);
-
- } catch (CommunicationException e) {
-
- // log the exception so we know it's there.
- containerLog.info(sm.getString("jndiRealm.exception.retry"), e);
-
- // close the connection so we know it will be reopened.
- if (context != null)
- close(context);
-
- // open a new directory context.
- context = open();
-
- // Try the authentication again.
- principal = authenticate(context, username, credentials);
-
- } catch (ServiceUnavailableException e) {
-
- // log the exception so we know it's there.
- containerLog.info(sm.getString("jndiRealm.exception.retry"), e);
-
- // close the connection so we know it will be reopened.
- if (context != null)
- close(context);
-
- // open a new directory context.
- context = open();
-
- // Try the authentication again.
- principal = authenticate(context, username, credentials);
-
}
@@ -2246,22 +2216,7 @@ public class JNDIRealm extends RealmBase
// Authenticate the specified username if possible
principal = getPrincipal(context, username, gssCredential);
- } catch (CommunicationException e) {
-
- // log the exception so we know it's there.
- containerLog.info(sm.getString("jndiRealm.exception.retry"), e);
-
- // close the connection so we know it will be reopened.
- if (context != null)
- close(context);
-
- // open a new directory context.
- context = open();
-
- // Try the authentication again.
- principal = getPrincipal(context, username, gssCredential);
-
- } catch (ServiceUnavailableException e) {
+ } catch (CommunicationException | ServiceUnavailableException e) {
// log the exception so we know it's there.
containerLog.info(sm.getString("jndiRealm.exception.retry"), e);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org