You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2011/07/18 18:26:33 UTC

svn commit: r1147954 - in /camel/trunk/components: camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java camel-http4/src/main/java/org/apache/camel/component/http4/HttpProducer.java

Author: davsclaus
Date: Mon Jul 18 16:26:32 2011
New Revision: 1147954

URL: http://svn.apache.org/viewvc?rev=1147954&view=rev
Log:
CAMEL-4242: Improved error message if bridging endpoints and end user have not enabled the bridgeEndpoint option.

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

Modified: camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java?rev=1147954&r1=1147953&r2=1147954&view=diff
==============================================================================
--- camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java (original)
+++ camel/trunk/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java Mon Jul 18 16:26:32 2011
@@ -344,6 +344,12 @@ public class HttpProducer extends Defaul
             }
         }
 
+        // there must be a host on the method
+        if (method.getHostConfiguration().getHost() == null) {
+            throw new IllegalArgumentException("Invalid uri: " + url
+                    + ". If you are forwarding/bridging http endpoints, then enable the bridgeEndpoint option on the endpoint: " + getEndpoint());
+        }
+
         return method;
     }
 

Modified: camel/trunk/components/camel-http4/src/main/java/org/apache/camel/component/http4/HttpProducer.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-http4/src/main/java/org/apache/camel/component/http4/HttpProducer.java?rev=1147954&r1=1147953&r2=1147954&view=diff
==============================================================================
--- camel/trunk/components/camel-http4/src/main/java/org/apache/camel/component/http4/HttpProducer.java (original)
+++ camel/trunk/components/camel-http4/src/main/java/org/apache/camel/component/http4/HttpProducer.java Mon Jul 18 16:26:32 2011
@@ -333,6 +333,11 @@ public class HttpProducer extends Defaul
             queryString = getEndpoint().getHttpUri().getRawQuery();
         }
 
+        if (uri.getScheme() == null || uri.getHost() == null) {
+            throw new IllegalArgumentException("Invalid uri: " + uri
+                    + ". If you are forwarding/bridging http endpoints, then enable the bridgeEndpoint option on the endpoint: " + getEndpoint());
+        }
+
         StringBuilder builder = new StringBuilder(uri.getScheme()).append("://").append(uri.getHost());
 
         if (uri.getPort() != -1) {