RFR: JAXP 1.5 for JDK8/7u40: 8016648: FEATURE_SECURE_PROCESSING set to true or false causes SAXParseException to be thrown

[Joe Darcy]

If you just want to test for platform version (7 vs 8), you might also 
look at javax.lang.model.SourceVersion.latest().

-Joe

On 7/1/2013 8:44 AM, Alan Bateman wrote:
> On 01/07/2013 09:09, huizhe wang wrote:
>> Hi,
>>
>> There have been two revisions in JAXP 1.5 specification: 1)  the 
>> relationship between the JAXP 1.5 properties and 
>> FEATURE_SECURE_PROCESSING (FSP) is now a recommendation. It is up to 
>> the implementation to decide if it wants to restrict when FSP is 
>> set.  2) System properties will override that may have been set by FSP.
>>
>> In the following patch, a JDK version detection code is added so that 
>> when FSP is set explicitly, for JDK7, there will be no restrictions, 
>> but for JDK8 and above,  the restrictions are on.
>>
>> The effective order is changed so that JAXP 1.5 defined system 
>> properties will override that may be set by enabling FSP.
>>
>> Please review:
>> http://cr.openjdk.java.net/~joehw/jdk8/8016648/webrev/
>>
>> Note that the patch is identical for JDK7 and 8, and I plan to ask 
>> approval for 7u40 integration.
>>
>> Thanks,
>> Joe
>
> Are there tests that can be used to verify this (both for 8 and 7u40? 
> Are you confident that tests cover all the overrides scenarios?
>
> I think isJDKandAbove has the assume the long standing format for 
> java.version. If someone really did change the format to what you are 
> suggesting then the code would fail with a value such as "7.40".
>
> In getJAXPSystemProperty's comment then I assume it should be 
> ${java.home}.
>
> -Alan
>
>
>