You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@camel.apache.org by ni...@apache.org on 2009/12/16 13:09:54 UTC

svn commit: r891208 - /camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java

Author: ningjiang
Date: Wed Dec 16 12:09:53 2009
New Revision: 891208

URL: http://svn.apache.org/viewvc?rev=891208&view=rev
Log:
CAMEL-2293 Checked the empty string as Charles suggested.

Modified:
    camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java

Modified: camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java?rev=891208&r1=891207&r2=891208&view=diff
==============================================================================
--- camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java (original)
+++ camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java Wed Dec 16 12:09:53 2009
@@ -88,7 +88,7 @@
         HttpClient answer = new HttpClient(getClientParams());
 
         // configure http proxy from camelContext
-        if (getCamelContext().getProperties().get("http.proxyHost") != null && getCamelContext().getProperties().get("http.proxyPort") != null) {
+        if (ObjectHelper.isNotEmpty(getCamelContext().getProperties().get("http.proxyHost")) && ObjectHelper.isNotEmpty(getCamelContext().getProperties().get("http.proxyPort"))) {
             String host = getCamelContext().getProperties().get("http.proxyHost");
             int port = Integer.parseInt(getCamelContext().getProperties().get("http.proxyPort"));
             if (LOG.isDebugEnabled()) {