[
http://jira.jboss.com/jira/browse/JBAS-1955?page=all ]
Dimitris Andreadis reopened JBAS-1955:
--------------------------------------
Magesh, I don't agree with your changes. If you read through my comments you'll
see I only wanted to deal with the transfer of the MBeanInfo, not the getAttributes call.
And even if I wanted to deal with getAttributes I would do so by creating a new policy,
not the changing StripModleMBeanInfoPolicy that does what it's name says.
We are also very close to a code freeze state, so no new features at this point.
Please rollback (and keep a local copy of) your changes. We'll reconsider them for a
later release, after discussing the issue in the forums.
XMBean Interceptor for InvokerAdaptorService to deal with
NonSerializableExceptions
-----------------------------------------------------------------------------------
Key: JBAS-1955
URL:
http://jira.jboss.com/jira/browse/JBAS-1955
Project: JBoss Application Server
Issue Type: Feature Request
Security Level: Public(Everyone can see)
Components: Management services
Affects Versions: JBossAS-4.0.2 Final
Environment: jmx, twiddle
Reporter: Fabiano C. de Oliveira
Assigned To: Magesh Kumar B
Fix For: JBossAS-4.0.5.SP1 , JBossAS-4.2.0.GA, JBossAS-5.0.0.Beta2
Attachments: JBAS-1955a.zip
Time Spent: 3 days, 3 hours
Remaining Estimate: 0 minutes
This patch add a Interceptor to org.jboss.jmx.connector.invoker.InvokerAdaptorService
besides AuthenticationInterceptor that is responsable for prevent remote clients to launch
NonSerializable exception.
The interception have 3 types of action: Invisible, Null and Wrapper.
In the Invisible mode all NonSerializable fields are ignored so if you call getMBeanInfo
all NonSerializable will be remove and returned to your remote client(twiddle, MC4J, etc).
This mode is more tested so it is recommended.
In the null mode NonSerializable fields are visible but always return Null.
The wraper mode is not implemented, but the idea is replace the NonSerializable Class for
another
The interceptor is only a class. The tests are in the patch too. Only test for invisible
mode. I´m still doing tests for the other modes.
Any idea is very appreciated. I dont have sufficient time to work more in this code. But
I hope to do soon.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira