[RFR] 8205525 : Improve exception messages during manifest parsing of jar archives
Alan Bateman
Alan.Bateman at oracle.com
Tue Jul 17 11:39:02 UTC 2018
On 16/07/2018 14:53, Baesken, Matthias wrote:
> Hello, after latest comments from Alan and Jaikiran I created a new webrev :
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8205525.2/
>
> The jar file path is now printed in case jdk.includeInExceptions contains jarpath (this approach is "borrowed" from the enhanced socket exceptions ) .
> The line number is always printed .
>
The general approach seems okay and consistent with the agreement on how
to reveal host names in socket exceptions.
The name of the security/system property will need discussion as
"jdk.includeInExceptions" is very general. If we have something general
then we'll need a good name and replace the existing
jdk.net.includeInExceptions. It might be better to go with something
specific for the area such as "jdk.util.jar.includeInExceptions=jarpath"
(to be consistent with other jdk.* properties in this code). A CSR will
be needed for this too as the property will be documented in the
java.security file.
As regards the patch then it mostly looks okay but I think the changes
in Attributes will need cleanup to get it consistent (esp. the line
lengths) with the existing code.
-Alan
More information about the core-libs-dev
mailing list