directory-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From erodrig...@apache.org
Subject 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
Date Sat, 29 Oct 2005 03:10:08 GMT
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;
         }
 



Mime
View raw message