[netcdf-java] Small bug in HttpClientManager

[Jon Blower]

Hi all (particularly John C),

I've spotted a small bug in HttpClientManager in the section below:

    // nick.bower at metoceanengineers.com
    String proxyHost = System.getProperty("http.proxyHost");
    String proxyPort = System.getProperty("http.proxyPort");
    if ((proxyHost != null) && (proxyPort != null)) {
        _client.getHostConfiguration().setProxy(proxyHost,
Integer.parseInt(proxyPort));
    }

The problem here is that if the "http.proxyPort" property is set to
the empty string, the call to Integer.parseInt() fails and the class
doesn't get initialized.  (For some reason http.proxyPort is set to ""
on my system instead of null - don't know why, I'm not doing this
consciously.)

I suggest changing the if clause to:

    if ((proxyHost != null) && (proxyPort != null) &&
!proxyPort.trim().equals("")) {

Cheers, Jon

-- 
--------------------------------------------------------------
Dr Jon Blower              Tel: +44 118 378 5213 (direct line)
Technical Director         Tel: +44 118 378 8741 (ESSC)
Reading e-Science Centre   Fax: +44 118 378 6413
ESSC                       Email: jdb at mail.nerc-essc.ac.uk
University of Reading
3 Earley Gate
Reading RG6 6AL, UK
--------------------------------------------------------------