RFR 8056174: New APIs for jar signing

Sean Mullan sean.mullan at oracle.com
Mon Sep 28 19:27:21 UTC 2015


Looks good, just a couple of comments:

AlgorithmId: can you use braces around the conditional statements on 
lines 1008-1017?

Function: are you missing an @modules tag for the jarsigner module?

Options.java: why not use the JarSigner API here instead of the 
jarsigner tool?

--Sean

On 09/21/2015 11:43 PM, Weijun Wang wrote:
> Missing from the webrev, some extra lines needed in modules.xml of the
> parent repo jdk9/dev:
>
>     <module>
>       <name>jdk.jartool</name>
>       <depend>java.base</depend>
>       <export>
>         <name>com.sun.jarsigner</name>
>       </export>
> +    <export>
> +      <name>jdk.security.jarsigner</name>
> +    </export>
>     </module>
>
> Thanks
> Max
>
> On 09/22/2015 09:12 AM, Weijun Wang wrote:
>> OK, webrev updated at
>>
>>    http://cr.openjdk.java.net/~weijun/8056174/webrev.05/
>>
>> A new method setProperty() is added to JarSigner.Builder (as well as
>> getProperty() in JarSigner) to cover all misc settings. JarSignerX and
>> BuilderX are removed now.
>>
>> Thanks
>> Max
>>



More information about the security-dev mailing list