You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2010/04/01 02:30:55 UTC

svn commit: r929757 - in /geronimo/server/trunk/plugins/client: client/src/main/resources/OSGI-INF/blueprint/naming-providers.xml geronimo-client/src/main/java/org/apache/geronimo/client/AppClientContainer.java

Author: djencks
Date: Thu Apr  1 00:30:55 2010
New Revision: 929757

URL: http://svn.apache.org/viewvc?rev=929757&view=rev
Log:
GERONIMO-4934 enable openejb remote jndi, fix a lookup glitch in app client container

Modified:
    geronimo/server/trunk/plugins/client/client/src/main/resources/OSGI-INF/blueprint/naming-providers.xml
    geronimo/server/trunk/plugins/client/geronimo-client/src/main/java/org/apache/geronimo/client/AppClientContainer.java

Modified: geronimo/server/trunk/plugins/client/client/src/main/resources/OSGI-INF/blueprint/naming-providers.xml
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/client/client/src/main/resources/OSGI-INF/blueprint/naming-providers.xml?rev=929757&r1=929756&r2=929757&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/client/client/src/main/resources/OSGI-INF/blueprint/naming-providers.xml (original)
+++ geronimo/server/trunk/plugins/client/client/src/main/resources/OSGI-INF/blueprint/naming-providers.xml Thu Apr  1 00:30:55 2010
@@ -72,5 +72,9 @@ limitations under the License.
         <bean class="com.sun.jndi.url.rmi.rmiURLContextFactory"/>
     </service>
 
+    <service id="OpenEJBInitialContextFactory"
+             interface="org.apache.openejb.client.RemoteInitialContextFactory">
+        <bean class="org.apache.openejb.client.RemoteInitialContextFactory"/>
+    </service>
 
 </blueprint>
\ No newline at end of file

Modified: geronimo/server/trunk/plugins/client/geronimo-client/src/main/java/org/apache/geronimo/client/AppClientContainer.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/client/geronimo-client/src/main/java/org/apache/geronimo/client/AppClientContainer.java?rev=929757&r1=929756&r2=929757&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/client/geronimo-client/src/main/java/org/apache/geronimo/client/AppClientContainer.java (original)
+++ geronimo/server/trunk/plugins/client/geronimo-client/src/main/java/org/apache/geronimo/client/AppClientContainer.java Thu Apr  1 00:30:55 2010
@@ -152,8 +152,9 @@ public final class AppClientContainer im
                 for (Injection injection : injections) {
                     try {
                         String jndiName = injection.getJndiName();
-                        //our componentContext is attached to jndi at "java:comp" so we remove that when looking stuff up in it
-                        Object object = componentContext.lookup("env/" + jndiName);
+                        //TODO this may not be correct due to module, app, and global jndi names.
+                        //our componentContext is attached to jndi at "java:" so we remove that when looking stuff up in it
+                        Object object = componentContext.lookup("comp/env/" + jndiName);
                         if (object instanceof String) {
                             String string = (String) object;
                             // Pass it in raw so it could be potentially converted to