Re: RFR 6563286 6797318 8177648 - Undeclared IAE thrown from HttpURLConnection.connect for some URLs

2019-08-01 Thread Jaikiran Pai
Hello Michael, On 01/08/19 4:22 PM, Michael McMahon wrote: > Hi Jaikiran, > > This looks good to me. Thank you. > There are a number of other places where ProxySelector::select > gets called which could benefit from the same change. Personally, I > think it would be okay > to deal with those i

Re: RFR 6563286 6797318 8177648 - Undeclared IAE thrown from HttpURLConnection.connect for some URLs

2019-08-01 Thread Michael McMahon
Hi Jaikiran, This looks good to me. There are a number of other places where ProxySelector::select gets called which could benefit from the same change. Personally, I think it would be okay to deal with those in a followup issue, but maybe others have another view on that. I can create the CSR

Re: RFR 6563286 6797318 8177648 - Undeclared IAE thrown from HttpURLConnection.connect for some URLs

2019-08-01 Thread Jaikiran Pai
Hello Michael, On 31/07/19 11:43 AM, Michael McMahon wrote: > ... >>> >>> I wonder if another solution is possible where the IAE is caught >>> at the appropriate place(s) and converted to an IOException >>> which is what users are expecting. >> I can update the proposed patch to redo it to catch t