RFR: 8043129: JAF initialisation in SAAJ clashing with the one in javax.mail
chris.hegarty at oracle.com
Tue May 20 13:15:17 UTC 2014
On 05/20/2014 11:07 AM, Miroslav Kos wrote:
> Hi Chris,
> thanks for pointing - I forgot to add copyright. Will fix.
> Regarding jar attachment - this is a slightly modified version of test
> as received from customer. If there is any mechanism allowing to use
> maven artifact (I am not aware of it) with jprt, it would be definitelly
> better solution, but I didn't find it.
> I could rewrite the test that it would test just not adding this
> handler, but it wouldn't explain much what the issue was about - I could
> write it to javadoc?
> Let me know, what you think.
I see we have many other jar files under jdk/test, so maybe this is ok.
> On 19/05/14 19:27, Chris Hegarty wrote:
>> The source change looks ok to me. Have you commented out the line
>> adding the handler because it may be reinstated, or as a reminder that
>> it was deliberately omitted?
>> The test, at the very minimum, will need a suitable copyright header.
>> I also wonder if there is any other way of testing this issue without
>> adding, what appears to be, a large jar file.
>> On 19/05/14 14:36, Miroslav Kos wrote:
>>> this is review request for:
>>> 8043129: JAF initialisation in SAAJ clashing with the one in javax.mail
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8043129
>>> http://cr.openjdk.java.net/~mkos/8043129/jdk.00/ (test)
>>> This solves conflict between JAF handlers for JAX-WS and javax.mail. The
>>> issue has been introduced in the latest CPU.
More information about the core-libs-dev