[jira] Updated: (FELIX-2270) Don't install jaas.boot bundle, its in the startup classpath

["David Jencks (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/FELIX-2270?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Jencks updated FELIX-2270:
--------------------------------

    Attachment: FELIX-2270.patch

Also, I suspect you want to use the copy of options without the ProxyLoginModule options to initialize the delegate login module.  Otherwise there's no reason to copy the options:

Index: jaas/boot/src/main/java/org/apache/felix/karaf/jaas/boot/ProxyLoginModule.java
===================================================================
--- jaas/boot/src/main/java/org/apache/felix/karaf/jaas/boot/ProxyLoginModule.java      (revision 932189)
+++ jaas/boot/src/main/java/org/apache/felix/karaf/jaas/boot/ProxyLoginModule.java      (working copy)
@@ -73,7 +73,7 @@
         } catch (Exception e) {
             throw new IllegalStateException("Can not load or create login module " + module + " for bundle " + bundleId, e);
         }
-        target.initialize(subject, callbackHandler, sharedState, options);
+        target.initialize(subject, callbackHandler, sharedState, newOptions);
     }
 
     /* (non-Javadoc)

> Don't install jaas.boot bundle, its in the startup classpath
> ------------------------------------------------------------
>
>                 Key: FELIX-2270
>                 URL: https://issues.apache.org/jira/browse/FELIX-2270
>             Project: Felix
>          Issue Type: Bug
>          Components: Karaf
>    Affects Versions: karaf-1.4.0
>            Reporter: David Jencks
>            Assignee: Guillaume Nodet
>             Fix For: karaf 1.6.0
>
>         Attachments: FELIX-2270.patch
>
>
> etc/startup.properties should not have the line for jaas.boot bundle, as it is in the startup classloader for karaf and only works there.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.