You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@karaf.apache.org by gn...@apache.org on 2012/11/09 15:09:22 UTC

svn commit: r1407462 - /karaf/branches/karaf-2.3.x/jaas/authz/src/main/java/org/apache/karaf/jaas/authz/impl/AuthorizationServiceImpl.java

Author: gnodet
Date: Fri Nov  9 14:09:22 2012
New Revision: 1407462

URL: http://svn.apache.org/viewvc?rev=1407462&view=rev
Log:
[KARAF-979] Fix integration tests failures, allow null Subject in authorization

Modified:
    karaf/branches/karaf-2.3.x/jaas/authz/src/main/java/org/apache/karaf/jaas/authz/impl/AuthorizationServiceImpl.java

Modified: karaf/branches/karaf-2.3.x/jaas/authz/src/main/java/org/apache/karaf/jaas/authz/impl/AuthorizationServiceImpl.java
URL: http://svn.apache.org/viewvc/karaf/branches/karaf-2.3.x/jaas/authz/src/main/java/org/apache/karaf/jaas/authz/impl/AuthorizationServiceImpl.java?rev=1407462&r1=1407461&r2=1407462&view=diff
==============================================================================
--- karaf/branches/karaf-2.3.x/jaas/authz/src/main/java/org/apache/karaf/jaas/authz/impl/AuthorizationServiceImpl.java (original)
+++ karaf/branches/karaf-2.3.x/jaas/authz/src/main/java/org/apache/karaf/jaas/authz/impl/AuthorizationServiceImpl.java Fri Nov  9 14:09:22 2012
@@ -67,8 +67,10 @@ public class AuthorizationServiceImpl im
 
     public List<String> getPrincipals(Subject subject) {
         List<String> principals = new ArrayList<String>();
-        for (Principal principal : subject.getPrincipals()) {
-            principals.add(principal.getClass().getName() + ":" + principal.getName());
+        if (subject != null) {
+            for (Principal principal : subject.getPrincipals()) {
+                principals.add(principal.getClass().getName() + ":" + principal.getName());
+            }
         }
         return principals;
     }