You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by le...@apache.org on 2007/10/24 08:23:35 UTC

svn commit: r587798 - in /harmony/enhanced/classlib/trunk/modules/auth: META-INF/ src/main/java/common/org/apache/harmony/auth/jgss/kerberos/ src/main/java/common/org/apache/harmony/auth/jgss/kerberos/toolbox/

Author: leoli
Date: Tue Oct 23 23:23:32 2007
New Revision: 587798

URL: http://svn.apache.org/viewvc?rev=587798&view=rev
Log:
Apply patch for HARMONY-4721([classlib][auth]Harmony lacks default JGSS provider)(3) Refractor in org.apache.harmony.auth.jgss.kerberos.KerberosUtils & org.apache.harmony.auth.jgss.kerberos.toolbox.KerberosToolboxImpl.

Modified:
    harmony/enhanced/classlib/trunk/modules/auth/META-INF/MANIFEST.MF
    harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/KerberosUtils.java
    harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/toolbox/KerberosToolboxImpl.java

Modified: harmony/enhanced/classlib/trunk/modules/auth/META-INF/MANIFEST.MF
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/auth/META-INF/MANIFEST.MF?rev=587798&r1=587797&r2=587798&view=diff
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/auth/META-INF/MANIFEST.MF (original)
+++ harmony/enhanced/classlib/trunk/modules/auth/META-INF/MANIFEST.MF Tue Oct 23 23:23:32 2007
@@ -32,6 +32,7 @@
  javax.naming,
  javax.naming.directory,
  org.apache.harmony.kernel.vm,
+ org.apache.harmony.luni.util,
  org.apache.harmony.security,
  org.apache.harmony.security.asn1,
  org.apache.harmony.security.fortress,

Modified: harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/KerberosUtils.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/KerberosUtils.java?rev=587798&r1=587797&r2=587798&view=diff
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/KerberosUtils.java (original)
+++ harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/KerberosUtils.java Tue Oct 23 23:23:32 2007
@@ -163,5 +163,6 @@
             return tgt;
         }
         return getTGTFromLoginModule(clientPrincipal);
+        //TODO CACHE : Whether should attach this tgt to the subject for current AccessControlContext?
     }
 }

Modified: harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/toolbox/KerberosToolboxImpl.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/toolbox/KerberosToolboxImpl.java?rev=587798&r1=587797&r2=587798&view=diff
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/toolbox/KerberosToolboxImpl.java (original)
+++ harmony/enhanced/classlib/trunk/modules/auth/src/main/java/common/org/apache/harmony/auth/jgss/kerberos/toolbox/KerberosToolboxImpl.java Tue Oct 23 23:23:32 2007
@@ -19,6 +19,8 @@
 
 import javax.security.auth.kerberos.KerberosTicket;
 
+import org.apache.harmony.luni.util.NotImplementedException;
+
 
 /*
  * The class will wrap the dependency on external kerberos tools.
@@ -33,12 +35,12 @@
     
     public KerberosTicket getTGS(String serverPrincipalName, KerberosTicket TGT) {
         // TODO Auto-generated method stub
-        return null;
+        throw new NotImplementedException();
     }
 
     public KerberosTicket getTGT(String clientPrincipalName, char[] password) {
         // TODO Auto-generated method stub
-        return null;
+        throw new NotImplementedException();
     }
 
 }