Re: [7u8] [Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-10 Thread Paul Sandoz
Hi Joe, What happens when someone logs a bug for system IDs containing IPv6 addresses and non-percent encoded international characters? On Jul 10, 2012, at 3:42 AM, Joe Wang wrote: Hi Paul, I'm back from vacation. You're right. But such an error is also expected. The original design

Re: [7u8] [Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-10 Thread Joe Wang
On 7/9/2012 10:59 PM, Paul Sandoz wrote: Hi Joe, What happens when someone logs a bug for system IDs containing IPv6 addresses and non-percent encoded international characters? Exception would be expected just as if Xerces is used. On Jul 10, 2012, at 3:42 AM, Joe Wang wrote: Hi Paul,

Re: [7u8] [Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-10 Thread Paul Sandoz
Hi Joe, If you are gonna fix things to support IPv6 addresses in URLs i really think you need to make it work for URLs with international characters too. On Jul 10, 2012, at 8:50 PM, Joe Wang wrote: 602 if (reader == null) { 603 stream = xmlInputSource.getByteStream();

Re: [7u8] [Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-10 Thread Joe Wang
Hi Paul, I agree, it looked quite messy, isn't it :), keeping a hack that generates encoded url and yet not support IPv6 addresses that contain international characters? It IS a more preferable solution to remove the hack of using escapeNonUSAscii -- so I'll do :) Thanks, Joe On

[7u8] [Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-09 Thread Joe Wang
Hi Paul, I'm back from vacation. You're right. But such an error is also expected. The original design never tried to out-do the java.net.URL. If a system ID input fails URL, it shall result in an exception. The patch that supplied the extra encoding was provided to both Sun and Apache,

Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-06-27 Thread Paul Sandoz
Hi, On Jun 26, 2012, at 11:59 PM, Joe Wang wrote: Hi Paul, That method was contributed by engineers from Korea and intended to handle paths that contained international characters, so that was how it was named. It was an extra processing added. Outside of that scenario, we'd want to

Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-06-26 Thread Joe Wang
Hi Paul, That method was contributed by engineers from Korea and intended to handle paths that contained international characters, so that was how it was named. It was an extra processing added. Outside of that scenario, we'd want to skip the process and get back to letting URL handle the

Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-06-25 Thread Paul Sandoz
Hi Joe, What happens if there is a space character or other characters in the string that should be encoded ? http://greenbytes.de/tech/webdav/rfc2396.html#rfc.section.2.4.3 I suspect escapeNonUSAscii is slightly misleading, it should be really called something like