JBoss Remoting SVN: r5286 - remoting2/branches/2.2/src/tests/org/jboss/test/remoting/marshall/config.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 17:17:51 -0400 (Tue, 19 May 2009)
New Revision: 5286
Modified:
remoting2/branches/2.2/src/tests/org/jboss/test/remoting/marshall/config/ConfigTestMarshaller.java
Log:
JBREM-1102: Added logging statement.
Modified: remoting2/branches/2.2/src/tests/org/jboss/test/remoting/marshall/config/ConfigTestMarshaller.java
===================================================================
--- remoting2/branches/2.2/src/tests/org/jboss/test/remoting/marshall/config/ConfigTestMarshaller.java 2009-05-19 21:13:05 UTC (rev 5285)
+++ remoting2/branches/2.2/src/tests/org/jboss/test/remoting/marshall/config/ConfigTestMarshaller.java 2009-05-19 21:17:51 UTC (rev 5286)
@@ -67,6 +67,7 @@
{
cloned = 0;
wrote = false;
+ log.info("reset: wrote: " + wrote + ", cloned: " + cloned);
}
}
14 years, 11 months
JBoss Remoting SVN: r5285 - remoting2/branches/2.x/docs/guide/en.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 17:13:05 -0400 (Tue, 19 May 2009)
New Revision: 5285
Modified:
remoting2/branches/2.x/docs/guide/en/chap17.xml
Log:
JBREM-1135: Updated release notes for 2.5.2.
Modified: remoting2/branches/2.x/docs/guide/en/chap17.xml
===================================================================
--- remoting2/branches/2.x/docs/guide/en/chap17.xml 2009-05-19 21:11:47 UTC (rev 5284)
+++ remoting2/branches/2.x/docs/guide/en/chap17.xml 2009-05-19 21:13:05 UTC (rev 5285)
@@ -77,17 +77,23 @@
<para>* [JBREM-1121] - Client SocketFactory should be configurable by InvokerLocator</para>
<para>* [JBREM-1125] - Test for IllegalStateException when calling Timer.schedule()</para>
<para>* [JBREM-1133] - CLONE [JBREM-1129] - Eliminate nondeterminism in Lease updates</para>
+ <para>* [JBREM-1137] - New jnpserver requires wrapping call to new InitialContext() in PrivilegedExceptionAction</para>
<para><emphasis role="bold">Feature request</emphasis></para>
+ <para>* [JBREM-1102] - Make configuration map available to MarshalFactory</para>
<para>* [JBREM-1123] - SocketServerInvoker needs an immediate shutdown option</para>
<para>* [JBREM-1124] - Invokers need option for configuration map parameters to override InvokerLocator parameters</para>
<para>* [JBREM-1132] - CLONE [JBREM-1128] - Introduce connection identity concept</para>
+ <para><emphasis role="bold">Release</emphasis></para>
+
+ <para>* [JBREM-1135] - Release 2.5.2</para>
+
<para><emphasis role="bold">Task</emphasis></para>
<para>* [JBREM-1134] - Assure version compatibility with earlier versions of Remoting</para>
- <para>* [JBREM-1135] - Release 2.5.2</para>
+ <para>* [JBREM-1136] - Update jars to match AS 5.1.0</para>
<bridgehead>Release Notes - JBoss Remoting - Version 2.5.1 (Flounder)</bridgehead>
14 years, 11 months
JBoss Remoting SVN: r5284 - remoting2/branches/2.x/docs.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 17:11:47 -0400 (Tue, 19 May 2009)
New Revision: 5284
Modified:
remoting2/branches/2.x/docs/README.txt
Log:
JBREM-1131: Added 2.5.2 release notes.
Modified: remoting2/branches/2.x/docs/README.txt
===================================================================
--- remoting2/branches/2.x/docs/README.txt 2009-05-19 21:09:58 UTC (rev 5283)
+++ remoting2/branches/2.x/docs/README.txt 2009-05-19 21:11:47 UTC (rev 5284)
@@ -41,6 +41,32 @@
==========================================================================================================
==========================================================================================================
+Release Notes - JBoss Remoting - Version 2.5.2 (Flounder)
+
+Bug
+
+ * [JBREM-1121] - Client SocketFactory should be configurable by InvokerLocator
+ * [JBREM-1125] - Test for IllegalStateException when calling Timer.schedule()
+ * [JBREM-1133] - CLONE [JBREM-1129] - Eliminate nondeterminism in Lease updates
+ * [JBREM-1137] - New jnpserver requires wrapping call to new InitialContext() in PrivilegedExceptionAction
+
+Feature Request
+
+ * [JBREM-1102] - Make configuration map available to MarshalFactory
+ * [JBREM-1123] - SocketServerInvoker needs an immediate shutdown option
+ * [JBREM-1124] - Invokers need option for configuration map parameters to override InvokerLocator parameters
+ * [JBREM-1132] - CLONE [JBREM-1128] - Introduce connection identity concept
+
+Release
+
+ * [JBREM-1135] - Release 2.5.2
+
+Task
+
+ * [JBREM-1134] - Assure version compatibility with earlier versions of Remoting
+ * [JBREM-1136] - Update jars to match AS 5.1.0
+
+==========================================================================================================
Release Notes - JBoss Remoting - Version 2.5.1 (Flounder) - Text format
Bug
14 years, 11 months
JBoss Remoting SVN: r5283 - remoting2/branches/2.2/docs.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 17:09:58 -0400 (Tue, 19 May 2009)
New Revision: 5283
Modified:
remoting2/branches/2.2/docs/README.txt
Log:
JBREM-1131: Added 2.2.2 release notes.
Modified: remoting2/branches/2.2/docs/README.txt
===================================================================
--- remoting2/branches/2.2/docs/README.txt 2009-05-19 20:53:07 UTC (rev 5282)
+++ remoting2/branches/2.2/docs/README.txt 2009-05-19 21:09:58 UTC (rev 5283)
@@ -27,6 +27,47 @@
in Jira, please create one.
==========================================================================================================
+Release Notes - JBoss Remoting - Version 2.2.3
+
+Bug
+
+ * [JBREM-1081] - Fix NPE in ServerInvokerCallbackHandler
+ * [JBREM-1088] - MicroSocketClientInvoker(InvokerLocator locator, Map configuration) ctor - not propagating exceptions (only message is wrapped)
+ * [JBREM-1099] - Make MulticastDetector detection message send buffer size configurable
+ * [JBREM-1109] - Eliminate race in MicroRemoteClientInvoker.getDataType()
+ * [JBREM-1111] - CLONE [JBREM-851] - In LeasePinger replace Timer if it has shut down
+ * [JBREM-1112] - Potential race between ConnectionValidator and ConnectionListener upon connection failure
+ * [JBREM-1119] - CLONE [JBREM-1113] - ServerInvokerCallbackHandlers leak when client doesn't shut down
+ * [JBREM-1121] - Client SocketFactory should be configurable by InvokerLocator
+ * [JBREM-1125] - Test for IllegalStateException when calling Timer.schedule()
+ * [JBREM-1127] - ClassCastExceptions arising from Unmarsharller/Classloader caching in MicroRemoteClientInvoker
+ * [JBREM-1129] - Eliminate nondeterminism in Lease updates
+
+Feature Request
+
+ * [JBREM-1082] - Allow ConnectionValidator to access InvokerLocator parameters
+ * [JBREM-1084] - Allow CallbackPoller to access Client and InvokerLocator parameters
+ * [JBREM-1100] - Link ServerInvokerServlet instances to Connectors via MBean names rather than locator URLs
+ * [JBREM-1102] - Make configuration map available to MarshalFactory
+ * [JBREM-1128] - Introduce connection identity concept
+
+Release
+
+ * [JBREM-1131] - Release 2.2.3
+
+Task
+
+ * [JBREM-139] - need automated test for servlet server invoker
+ * [JBREM-1078] - Remove reference to "numberOfRetries" parameter from Remoting Guide
+ * [JBREM-1079] - Port fixes to ServletServerInvoker from branch 2.x to branch 2.2
+ * [JBREM-1085] - Reduce log level of ServerSocketWrapper.close() log messages
+ * [JBREM-1103] - Correct javadoc for Client.invokeOneway()
+ * [JBREM-1104] - Identity.get() should create a more meaningful RuntimeException message
+ * [JBREM-1108] - Warn against making ConnectionValidator.validatorPingPeriod shorter than ConnectionValidator.validatorPingTimeout
+ * [JBREM-1110] - InvokerLocator.getParameters() should not return null
+ * [JBREM-1130] - Assure version compatibility with earlier versions of Remoting
+
+==========================================================================================================
Release Notes - JBoss Remoting - Version 2.2.2.SP11
Bug
14 years, 11 months
JBoss Remoting SVN: r5282 - remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/jboss.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 16:53:07 -0400 (Tue, 19 May 2009)
New Revision: 5282
Modified:
remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/jboss/JBossSerializationManager.java
Log:
JBREM-1127: Updates ObjectInputStream classloader if RemotingClassLoader's useCurrentThreadClassLoader variable is set to true.
Modified: remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/jboss/JBossSerializationManager.java
===================================================================
--- remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/jboss/JBossSerializationManager.java 2009-05-19 20:52:23 UTC (rev 5281)
+++ remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/jboss/JBossSerializationManager.java 2009-05-19 20:53:07 UTC (rev 5282)
@@ -25,6 +25,7 @@
import org.jboss.logging.Logger;
import org.jboss.remoting.loading.ObjectInputStreamWithClassLoader;
+import org.jboss.remoting.loading.RemotingClassLoader;
import org.jboss.remoting.serialization.IMarshalledValue;
import org.jboss.remoting.serialization.SerializationManager;
import org.jboss.remoting.serialization.SerializationStreamFactory;
@@ -95,18 +96,30 @@
{
ObjectInputStream objInputStream = null;
Object obj = null;
+ boolean useCurrentThreadClassLoader = false;
+
if (inputStream instanceof ObjectInputStreamWithClassLoader)
{
- if (((ObjectInputStreamWithClassLoader) inputStream).getClassLoader() == null)
+ ClassLoader cl = ((ObjectInputStreamWithClassLoader) inputStream).getClassLoader();
+ if (cl instanceof RemotingClassLoader)
{
+ useCurrentThreadClassLoader = ((RemotingClassLoader) cl).isUseCurrentThreadClassLoader();
+ }
+ if (cl == null || useCurrentThreadClassLoader)
+ {
((ObjectInputStreamWithClassLoader) inputStream).setClassLoader(customClassLoader);
}
objInputStream = (ObjectInputStream) inputStream;
}
else if (inputStream instanceof JBossObjectInputStream)
{
- if (((JBossObjectInputStream) inputStream).getClassLoader() == null)
+ ClassLoader cl = ((JBossObjectInputStream) inputStream).getClassLoader();
+ if (cl instanceof RemotingClassLoader)
{
+ useCurrentThreadClassLoader = ((RemotingClassLoader) cl).isUseCurrentThreadClassLoader();
+ }
+ if (cl == null || useCurrentThreadClassLoader )
+ {
((JBossObjectInputStream) inputStream).setClassLoader(customClassLoader);
}
objInputStream = (ObjectInputStream) inputStream;
14 years, 11 months
JBoss Remoting SVN: r5281 - remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/java.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 16:52:23 -0400 (Tue, 19 May 2009)
New Revision: 5281
Modified:
remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/java/JavaSerializationManager.java
Log:
JBREM-1127: Updates ObjectInputStream classloader if RemotingClassLoader's useCurrentThreadClassLoader variable is set to true.
Modified: remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/java/JavaSerializationManager.java
===================================================================
--- remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/java/JavaSerializationManager.java 2009-05-19 20:51:11 UTC (rev 5280)
+++ remoting2/branches/2.2/src/main/org/jboss/remoting/serialization/impl/java/JavaSerializationManager.java 2009-05-19 20:52:23 UTC (rev 5281)
@@ -31,6 +31,7 @@
import org.jboss.logging.Logger;
import org.jboss.remoting.Version;
import org.jboss.remoting.loading.ObjectInputStreamWithClassLoader;
+import org.jboss.remoting.loading.RemotingClassLoader;
import org.jboss.remoting.serialization.SerializationManager;
import org.jboss.remoting.serialization.SerializationStreamFactory;
import org.jboss.remoting.serialization.IMarshalledValue;
@@ -204,8 +205,14 @@
Object obj = null;
if(inputStream instanceof ObjectInputStreamWithClassLoader)
{
- if(((ObjectInputStreamWithClassLoader) inputStream).getClassLoader() == null)
+ ClassLoader cl = ((ObjectInputStreamWithClassLoader) inputStream).getClassLoader();
+ boolean useCurrentThreadClassLoader = false;
+ if (cl instanceof RemotingClassLoader)
{
+ useCurrentThreadClassLoader = ((RemotingClassLoader) cl).isUseCurrentThreadClassLoader();
+ }
+ if(cl == null || useCurrentThreadClassLoader)
+ {
((ObjectInputStreamWithClassLoader) inputStream).setClassLoader(customClassLoader);
}
objInputStream = (ObjectInputStream) inputStream;
14 years, 11 months
JBoss Remoting SVN: r5280 - remoting2/branches/2.2/src/main/org/jboss/remoting/loading.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 16:51:11 -0400 (Tue, 19 May 2009)
New Revision: 5280
Modified:
remoting2/branches/2.2/src/main/org/jboss/remoting/loading/RemotingClassLoader.java
Log:
JBREM-1127: Added useCurrentThreadClassLoader variable.
Modified: remoting2/branches/2.2/src/main/org/jboss/remoting/loading/RemotingClassLoader.java
===================================================================
--- remoting2/branches/2.2/src/main/org/jboss/remoting/loading/RemotingClassLoader.java 2009-05-19 20:50:24 UTC (rev 5279)
+++ remoting2/branches/2.2/src/main/org/jboss/remoting/loading/RemotingClassLoader.java 2009-05-19 20:51:11 UTC (rev 5280)
@@ -31,7 +31,8 @@
public class RemotingClassLoader extends ClassLoader
{
private ClassLoader secondaryClassLoader = null;
-
+ private boolean useCurrentThreadClassLoader;
+
protected static final Logger log = Logger.getLogger(RemotingClassLoader.class);
protected static final boolean isTrace = log.isTraceEnabled();
@@ -47,12 +48,7 @@
try
{
- ClassLoader primaryClassLoader = getParent();
- if (primaryClassLoader == null)
- {
- primaryClassLoader = Thread.currentThread().getContextClassLoader();
- }
- loadedClass = Class.forName(name, false, primaryClassLoader);
+ loadedClass = Class.forName(name, false, getParent());
if (isTrace) log.trace("loaded from primary classloader: " + name);
}
catch(ClassNotFoundException e)
@@ -86,4 +82,18 @@
return loadedClass;
}
+ public String toString()
+ {
+ return "RemotingClassLoader[" + System.identityHashCode(this) + ":" + getParent() + "]";
+ }
+
+ public boolean isUseCurrentThreadClassLoader()
+ {
+ return useCurrentThreadClassLoader;
+ }
+
+ public void setUseCurrentThreadClassLoader(boolean useCurrentThreadClassLoader)
+ {
+ this.useCurrentThreadClassLoader = useCurrentThreadClassLoader;
+ }
}
\ No newline at end of file
14 years, 11 months
JBoss Remoting SVN: r5279 - remoting2/branches/2.2/src/main/org/jboss/remoting.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 16:50:24 -0400 (Tue, 19 May 2009)
New Revision: 5279
Modified:
remoting2/branches/2.2/src/main/org/jboss/remoting/MicroRemoteClientInvoker.java
Log:
JBREM-1127: Changed use of useCurrentThreadClassLoader.
Modified: remoting2/branches/2.2/src/main/org/jboss/remoting/MicroRemoteClientInvoker.java
===================================================================
--- remoting2/branches/2.2/src/main/org/jboss/remoting/MicroRemoteClientInvoker.java 2009-05-19 20:30:24 UTC (rev 5278)
+++ remoting2/branches/2.2/src/main/org/jboss/remoting/MicroRemoteClientInvoker.java 2009-05-19 20:50:24 UTC (rev 5279)
@@ -94,30 +94,17 @@
// creating a new classloader containing the remoting class loader (for remote classloading)
// and the current thread's class loader. This allows to load remoting classes as well as
// user's classes.
- ClassLoader remotingClassLoader = null;
- if (useCurrentThreadClassLoader)
+ RemotingClassLoader remotingClassLoader = null;
+ if (parentFirstClassLoading)
{
- if (parentFirstClassLoading)
- {
- remotingClassLoader = new RemotingClassLoader(getClassLoader(), null);
- }
- else
- {
- remotingClassLoader = new RemotingClassLoader(null, getClassLoader());
- }
+ remotingClassLoader = new RemotingClassLoader(getClassLoader(), Thread.currentThread().getContextClassLoader());
}
else
{
- if (parentFirstClassLoading)
- {
- remotingClassLoader = new RemotingClassLoader(getClassLoader(), Thread.currentThread().getContextClassLoader());
- }
- else
- {
- remotingClassLoader = new RemotingClassLoader(Thread.currentThread().getContextClassLoader(), getClassLoader());
- }
+ remotingClassLoader = new RemotingClassLoader(Thread.currentThread().getContextClassLoader(), getClassLoader());
}
-
+ remotingClassLoader.setUseCurrentThreadClassLoader(useCurrentThreadClassLoader);
+
// try by locator (in case unmarshaller class name specified)
Map map = passConfigMapToMarshalFactory ? configuration : null;
unmarshaller = MarshalFactory.getUnMarshaller(getLocator(), getClassLoader(), map);
@@ -130,9 +117,13 @@
throw new InvalidMarshallingResource(
"Can not find a valid unmarshaller for data type: " + getDataType());
}
- setUnMarshaller(unmarshaller);
}
unmarshaller.setClassLoader(remotingClassLoader);
+
+ if (!useCurrentThreadClassLoader)
+ {
+ setUnMarshaller(unmarshaller);
+ }
}
// if raw, then send only param of invocation request
14 years, 11 months
JBoss Remoting SVN: r5278 - remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/ssl/timeout.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 16:30:24 -0400 (Tue, 19 May 2009)
New Revision: 5278
Removed:
remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/ssl/timeout/SSLSocketWriteTimeoutTestCase.java
Log:
JBREM-1120: Temporarily disabled test.
Deleted: remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/ssl/timeout/SSLSocketWriteTimeoutTestCase.java
===================================================================
--- remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/ssl/timeout/SSLSocketWriteTimeoutTestCase.java 2009-05-19 18:52:15 UTC (rev 5277)
+++ remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/ssl/timeout/SSLSocketWriteTimeoutTestCase.java 2009-05-19 20:30:24 UTC (rev 5278)
@@ -1,41 +0,0 @@
-/*
-* JBoss, Home of Professional Open Source
-* Copyright 2009, JBoss Inc., and individual contributors as indicated
-* by the @authors tag. See the copyright.txt in the distribution for a
-* full listing of individual contributors.
-*
-* This is free software; you can redistribute it and/or modify it
-* under the terms of the GNU Lesser General Public License as
-* published by the Free Software Foundation; either version 2.1 of
-* the License, or (at your option) any later version.
-*
-* This software is distributed in the hope that it will be useful,
-* but WITHOUT ANY WARRANTY; without even the implied warranty of
-* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-* Lesser General Public License for more details.
-*
-* You should have received a copy of the GNU Lesser General Public
-* License along with this software; if not, write to the Free
-* Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
-* 02110-1301 USA, or see the FSF site: http://www.fsf.org.
-*/
-package org.jboss.test.remoting.transport.socket.ssl.timeout;
-
-import org.jboss.test.remoting.transport.socket.timeout.SSLWriteTimeoutTestParent;
-
-/**
- * Unit tests for JBREM-1120.
- *
- * @author <a href="ron.sigal(a)jboss.com">Ron Sigal</a>
- * @version $Rev$
- * <p>
- * Copyright Apr 22, 2009
- * </p>
- */
-public class SSLSocketWriteTimeoutTestCase extends SSLWriteTimeoutTestParent
-{
- protected String getTransport()
- {
- return "sslsocket";
- }
-}
14 years, 11 months
JBoss Remoting SVN: r5277 - remoting2/branches/2.x/src/tests/org/jboss/test/remoting/detection/jndi.
by jboss-remoting-commits@lists.jboss.org
Author: ron.sigal(a)jboss.com
Date: 2009-05-19 14:52:15 -0400 (Tue, 19 May 2009)
New Revision: 5277
Modified:
remoting2/branches/2.x/src/tests/org/jboss/test/remoting/detection/jndi/CleanDetectionTestClient.java
Log:
JBREM-1137: Wrapped call to new InitialContext() in PrivilegedExceptionAction.
Modified: remoting2/branches/2.x/src/tests/org/jboss/test/remoting/detection/jndi/CleanDetectionTestClient.java
===================================================================
--- remoting2/branches/2.x/src/tests/org/jboss/test/remoting/detection/jndi/CleanDetectionTestClient.java 2009-05-19 18:51:16 UTC (rev 5276)
+++ remoting2/branches/2.x/src/tests/org/jboss/test/remoting/detection/jndi/CleanDetectionTestClient.java 2009-05-19 18:52:15 UTC (rev 5277)
@@ -156,7 +156,7 @@
env.put(Context.PROVIDER_URL, detectorHost + ":" + detectorPort);
env.put(Context.URL_PKG_PREFIXES, urlPackage);
- InitialContext initialContext = new InitialContext(env);
+ InitialContext initialContext = createContext(env);
String subContextName = JNDIDetector.DETECTION_SUBCONTEXT_NAME;
try
@@ -248,4 +248,27 @@
throw (NamingException) e.getCause();
}
}
+
+ static private InitialContext createContext(final Properties env) throws NamingException
+ {
+ if (SecurityUtility.skipAccessControl())
+ {
+ return new InitialContext(env);
+ }
+
+ try
+ {
+ return (InitialContext) AccessController.doPrivileged( new PrivilegedExceptionAction()
+ {
+ public Object run() throws Exception
+ {
+ return new InitialContext(env);
+ }
+ });
+ }
+ catch (PrivilegedActionException e)
+ {
+ throw (RuntimeException) e.getCause();
+ }
+ }
}
14 years, 11 months