You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by er...@apache.org on 2005/10/29 05:10:08 UTC
svn commit: r329361 - in
/directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store:
MultiBaseSearch.java SingleBaseSearch.java operations/ChangePassword.java
operations/GetPrincipal.java
Author: erodriguez
Date: Fri Oct 28 20:10:04 2005
New Revision: 329361
URL: http://svn.apache.org/viewcvs?rev=329361&view=rev
Log:
Exception handling clean-up in kerberos-common:
o Removed printStackTrace()'s
o Added root cause Throwable's to ConfigurationException constructors.
Modified:
directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/MultiBaseSearch.java
directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/SingleBaseSearch.java
directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/ChangePassword.java
directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/GetPrincipal.java
Modified: directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/MultiBaseSearch.java
URL: http://svn.apache.org/viewcvs/directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/MultiBaseSearch.java?rev=329361&r1=329360&r2=329361&view=diff
==============================================================================
--- directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/MultiBaseSearch.java (original)
+++ directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/MultiBaseSearch.java Fri Oct 28 20:10:04 2005
@@ -66,7 +66,7 @@
catch ( Exception e )
{
String message = "Failed to get catalog context " + (String) env.get( Context.PROVIDER_URL );
- throw new ConfigurationException( message );
+ throw new ConfigurationException( message, e );
}
}
@@ -82,7 +82,7 @@
catch ( NamingException ne )
{
String message = "Failed to get initial context " + (String) env.get( Context.PROVIDER_URL );
- throw new ConfigurationException( message );
+ throw new ConfigurationException( message, ne );
}
}
@@ -98,7 +98,7 @@
catch ( NamingException ne )
{
String message = "Failed to get initial context " + (String) env.get( Context.PROVIDER_URL );
- throw new ConfigurationException( message );
+ throw new ConfigurationException( message, ne );
}
}
Modified: directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/SingleBaseSearch.java
URL: http://svn.apache.org/viewcvs/directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/SingleBaseSearch.java?rev=329361&r1=329360&r2=329361&view=diff
==============================================================================
--- directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/SingleBaseSearch.java (original)
+++ directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/SingleBaseSearch.java Fri Oct 28 20:10:04 2005
@@ -56,7 +56,7 @@
catch ( NamingException ne )
{
String message = "Failed to get initial context " + (String) env.get( Context.PROVIDER_URL );
- throw new ConfigurationException( message );
+ throw new ConfigurationException( message, ne );
}
}
Modified: directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/ChangePassword.java
URL: http://svn.apache.org/viewcvs/directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/ChangePassword.java?rev=329361&r1=329360&r2=329361&view=diff
==============================================================================
--- directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/ChangePassword.java (original)
+++ directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/ChangePassword.java Fri Oct 28 20:10:04 2005
@@ -78,7 +78,6 @@
}
catch ( NamingException e )
{
- e.printStackTrace();
return null;
}
Modified: directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/GetPrincipal.java
URL: http://svn.apache.org/viewcvs/directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/GetPrincipal.java?rev=329361&r1=329360&r2=329361&view=diff
==============================================================================
--- directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/GetPrincipal.java (original)
+++ directory/shared/kerberos/trunk/common/src/java/org/apache/kerberos/store/operations/GetPrincipal.java Fri Oct 28 20:10:04 2005
@@ -90,8 +90,6 @@
}
catch ( NamingException e )
{
- e.printStackTrace();
-
return null;
}