RFR: 8043129: JAF initialisation in SAAJ clashing with the one in javax.mail

[Chris Hegarty]

On 05/20/2014 11:07 AM, Miroslav Kos wrote:
> Hi Chris,
> thanks for pointing - I forgot to add copyright. Will fix.

Thanks.

> 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.

-Chris.

>
> Thanks
> Miran
>
> On 19/05/14 19:27, Chris Hegarty wrote:
>> Miran,
>>
>> 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.
>>
>> -Chris.
>>
>> On 19/05/14 14:36, Miroslav Kos wrote:
>>> Hi,
>>> 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
>>> WEBREV:
>>> http://cr.openjdk.java.net/~mkos/8043129/jaxws.00/
>>> 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.
>>>
>>> Thanks
>>> Miran
>