RFR [11] 8207846: Generalize the jdk.net.includeInExceptions security property

Chris Hegarty chris.hegarty at oracle.com
Fri Jul 20 14:18:47 UTC 2018


Roger,

> On 20 Jul 2018, at 14:52, Roger Riggs <roger.riggs at oracle.com> wrote:
> 
> Hi Chris,
> 
> It is very unusual for the processing of system properties to do *any* parsing except for delimiters
> including removing spaces, etc.  It complicates the handling and sets a bad precedent
> that makes it more complex for users and developers to know how to set property values.
> The whitespace trimming should be removed.

The addition of the whitespace trimming was to clarify the intent of
the existing wording regarding whitespace. It was not intended to
set a precedent, good or bad.

Regardless of any subjective opinion, whitespace are allowable
within system property values, so it’s a matter of whether or not
we want to deal with them explicitly or just leave it as an
implementation detail.

I’m ok to drop the trimming if you feel strongly about it. But just
to note that this now diverges even more from the original.

-Chris.

> $.02, Roger
> 
>>   http://cr.openjdk.java.net/~chegar/8207846/webrev.00/
>> 
>> -Chris.
>> 
>> P.S. It appears that jtreg does not support quoted system property values
>> with spaces on the @run line. I’ll file an issue against jtreg for this.
>> 
> 



More information about the core-libs-dev mailing list