RFR: 8283093: JMX connections should default to using an ObjectInputFilter [v3]

Serguei Spitsyn sspitsyn at openjdk.org
Tue Oct 25 18:21:19 UTC 2022


On Wed, 19 Oct 2022 17:54:02 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Set the management.properties  "com.sun.management.jmxremote.serial.filter.pattern" value by default, to restrict types that can be deserialized.
>> 
>> Use the example value from the Core Libraries guide (see section 2. Serialization Filtering / Built-in Filters / Filters for JMX), plus Subject which is needed when using authentication.
>> 
>> The sun/management tests run OK with this change.  The existing test sun/management/jmxremote/startstop/JMXStartStopTest.java will fail if the filter specified is made too restrictive.
>
> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Additional test with command-line filter setting.

This looks good to me in general.
But I've posted a couple of questions.
If an RN is needed then it is better to review it before integration.
Thanks,
Serguei

test/jdk/javax/management/remote/mandatory/connection/DefaultAgentFilterTest.java line 309:

> 307:         try {
> 308:             // Add custom filter on command-line.
> 309:             testDefaultAgent(null, "-Dcom.sun.management.jmxremote.serial.filter.pattern=\"java.lang.*;java.math.BigInteger;java.math.BigDecimal;java.util.*;javax.management.openmbean.*;javax.management.ObjectName;java.rmi.MarshalledObject;javax.security.auth.Subject;DefaultAgentFilterTest$MyTestObject;!*\"");

Long line. Can we use string concatenation to break it?

-------------

PR: https://git.openjdk.org/jdk/pull/10507


More information about the serviceability-dev mailing list