[jdk17] RFR: JDK-8268826: Cleanup Override in Context-Specific Deserialization Filters [v6]

Brent Christian bchristi at openjdk.java.net
Fri Jun 25 16:52:09 UTC 2021


On Fri, 25 Jun 2021 14:54:45 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Remove the unnecessary special case "OVERRIDE" in jdk.serialFilterFactory property.
>> Fix description in the example of a filter allowing platform classes.
>> Suppress some warnings about use of SecurityManager in tests.
>
> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update logging of faults in jdk.serialFilterFactory to log only the exception message
>   Simplify the logging.properties to only the needed settings

Marked as reviewed by bchristi (Reviewer).

test/jdk/java/io/Serializable/serialFilter/SerialFilterFunctionTest.java line 49:

> 47:         // Enable logging
> 48:         System.setProperty("java.util.logging.config.file",
> 49:                 System.getProperty("test.src", ".") + "/logging.properties");

Is `System.setProperty()` needed if it's already being set with -D ?

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

PR: https://git.openjdk.java.net/jdk17/pull/85


More information about the core-libs-dev mailing list