camel-http-common - as a common module

Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/05019f6f
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/05019f6f
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/05019f6f

Branch: refs/heads/master
Commit: 05019f6f0d63447ca9f9e3bc22bd0671580d5433
Parents: ac1f1af
Author: Claus Ibsen <davscl...@apache.org>
Authored: Thu Jul 23 11:31:35 2015 +0200
Committer: Claus Ibsen <davscl...@apache.org>
Committed: Thu Jul 23 15:04:34 2015 +0200

----------------------------------------------------------------------
 components/camel-gae/pom.xml                    | 49 ++++++++++----------
 .../gae/bind/HttpBindingInvocationHandler.java  |  4 +-
 .../camel/component/gae/http/GHttpBinding.java  |  2 -
 .../component/gae/http/GHttpComponent.java      | 13 +-----
 .../camel/component/gae/http/GHttpEndpoint.java | 16 ++-----
 .../camel/component/gae/task/GTaskBinding.java  |  7 +--
 .../component/gae/task/GTaskComponent.java      | 15 ++----
 .../camel/component/gae/task/GTaskEndpoint.java | 12 ++---
 .../component/gae/task/GTaskBindingTest.java    |  2 +-
 9 files changed, 43 insertions(+), 77 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/pom.xml
----------------------------------------------------------------------
diff --git a/components/camel-gae/pom.xml b/components/camel-gae/pom.xml
index 981e395..3b537c5 100644
--- a/components/camel-gae/pom.xml
+++ b/components/camel-gae/pom.xml
@@ -15,7 +15,8 @@
   See the License for the specific language governing permissions and
   limitations under the License.
 -->
-<project xmlns="http://maven.apache.org/POM/4.0.0"; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; 
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/maven-v4_0_0.xsd";>
+<project xmlns="http://maven.apache.org/POM/4.0.0"; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/maven-v4_0_0.xsd";>
   <modelVersion>4.0.0</modelVersion>
 
   <parent>
@@ -30,18 +31,18 @@
   <description>Camel GAE support</description>
 
   <properties>
-        <camel.osgi.import.before.defaults>
-            javax.servlet.*;version="${servlet-version-range}"
-        </camel.osgi.import.before.defaults>
-       
<camel.osgi.export.pkg>org.apache.camel.component.gae.*</camel.osgi.export.pkg>
-       <camel.osgi.export.service>
-               org.apache.camel.spi.ComponentResolver;component=gauth,
-               org.apache.camel.spi.ComponentResolver;component=ghttp,
-               org.apache.camel.spi.ComponentResolver;component=ghttps,
-               org.apache.camel.spi.ComponentResolver;component=glogin,
-               org.apache.camel.spi.ComponentResolver;component=gmail,
-               org.apache.camel.spi.ComponentResolver;component=gtask
-       </camel.osgi.export.service>
+    <camel.osgi.import.before.defaults>
+      javax.servlet.*;version="${servlet-version-range}"
+    </camel.osgi.import.before.defaults>
+    
<camel.osgi.export.pkg>org.apache.camel.component.gae.*</camel.osgi.export.pkg>
+    <camel.osgi.export.service>
+      org.apache.camel.spi.ComponentResolver;component=gauth,
+      org.apache.camel.spi.ComponentResolver;component=ghttp,
+      org.apache.camel.spi.ComponentResolver;component=ghttps,
+      org.apache.camel.spi.ComponentResolver;component=glogin,
+      org.apache.camel.spi.ComponentResolver;component=gmail,
+      org.apache.camel.spi.ComponentResolver;component=gtask
+    </camel.osgi.export.service>
   </properties>
 
   <dependencies>
@@ -134,15 +135,15 @@
     </dependency>
   </dependencies>
 
-    <build>
-        <plugins>
-            <plugin>
-                <artifactId>maven-surefire-plugin</artifactId>
-                <configuration>
-                    <forkCount>1</forkCount>
-                   <reuseForks>false</reuseForks>
-                </configuration>
-            </plugin>
-        </plugins>
-    </build>
+  <build>
+    <plugins>
+      <plugin>
+        <artifactId>maven-surefire-plugin</artifactId>
+        <configuration>
+          <forkCount>1</forkCount>
+          <reuseForks>false</reuseForks>
+        </configuration>
+      </plugin>
+    </plugins>
+  </build>
 </project>

http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/src/main/java/org/apache/camel/component/gae/bind/HttpBindingInvocationHandler.java
----------------------------------------------------------------------
diff --git 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/bind/HttpBindingInvocationHandler.java
 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/bind/HttpBindingInvocationHandler.java
index 21328e5..e2229ea 100644
--- 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/bind/HttpBindingInvocationHandler.java
+++ 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/bind/HttpBindingInvocationHandler.java
@@ -21,8 +21,8 @@ import java.lang.reflect.Method;
 
 import org.apache.camel.Endpoint;
 import org.apache.camel.Exchange;
-import org.apache.camel.component.http.HttpBinding;
-import org.apache.camel.component.http.HttpMessage;
+import org.apache.camel.http.common.HttpBinding;
+import org.apache.camel.http.common.HttpMessage;
 
 /**
  * Post-processes {@link HttpBinding} invocations by delegating to an

http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpBinding.java
----------------------------------------------------------------------
diff --git 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpBinding.java
 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpBinding.java
index 1493edd..fd86339 100644
--- 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpBinding.java
+++ 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpBinding.java
@@ -20,7 +20,6 @@ import java.io.ByteArrayInputStream;
 import java.io.InputStream;
 import java.net.URL;
 import java.util.zip.GZIPInputStream;
-
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
@@ -28,7 +27,6 @@ import com.google.appengine.api.urlfetch.HTTPHeader;
 import com.google.appengine.api.urlfetch.HTTPMethod;
 import com.google.appengine.api.urlfetch.HTTPRequest;
 import com.google.appengine.api.urlfetch.HTTPResponse;
-
 import org.apache.camel.Exchange;
 import org.apache.camel.Message;
 import org.apache.camel.component.gae.bind.InboundBinding;

http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpComponent.java
----------------------------------------------------------------------
diff --git 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpComponent.java
 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpComponent.java
index 1aaa6b1..c5bb708 100644
--- 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpComponent.java
+++ 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpComponent.java
@@ -18,21 +18,16 @@ package org.apache.camel.component.gae.http;
 
 import java.net.URI;
 import java.util.Map;
-
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
 import com.google.appengine.api.urlfetch.HTTPRequest;
 import com.google.appengine.api.urlfetch.HTTPResponse;
-
 import org.apache.camel.Endpoint;
 import org.apache.camel.component.gae.bind.InboundBinding;
 import org.apache.camel.component.gae.bind.OutboundBinding;
-import org.apache.camel.component.http.HttpClientConfigurer;
 import org.apache.camel.component.servlet.ServletComponent;
 import org.apache.camel.component.servlet.ServletEndpoint;
-import org.apache.commons.httpclient.HttpConnectionManager;
-import org.apache.commons.httpclient.params.HttpClientParams;
 
 /**
  * The <a href="http://camel.apache.org/ghttp.html";>Google App Engine HTTP
@@ -65,11 +60,7 @@ public class GHttpComponent extends ServletComponent {
     }
 
     @Override
-    protected ServletEndpoint createServletEndpoint(String endpointUri,
-            ServletComponent component, URI httpUri, HttpClientParams params,
-            HttpConnectionManager httpConnectionManager,
-            HttpClientConfigurer clientConfigurer) throws Exception {
-        return new GHttpEndpoint(endpointUri, component, httpUri, params,
-                httpConnectionManager, clientConfigurer);
+    protected ServletEndpoint createServletEndpoint(String endpointUri, 
ServletComponent component, URI httpUri) throws Exception {
+        return new GHttpEndpoint(endpointUri, component, httpUri);
     }
 }

http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpEndpoint.java
----------------------------------------------------------------------
diff --git 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpEndpoint.java
 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpEndpoint.java
index 897a09b..75e3cf7 100644
--- 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpEndpoint.java
+++ 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/http/GHttpEndpoint.java
@@ -34,15 +34,12 @@ import 
org.apache.camel.component.gae.bind.HttpBindingInvocationHandler;
 import org.apache.camel.component.gae.bind.InboundBinding;
 import org.apache.camel.component.gae.bind.OutboundBinding;
 import org.apache.camel.component.gae.bind.OutboundBindingSupport;
-import org.apache.camel.component.http.HttpBinding;
-import org.apache.camel.component.http.HttpClientConfigurer;
 import org.apache.camel.component.servlet.ServletComponent;
 import org.apache.camel.component.servlet.ServletEndpoint;
+import org.apache.camel.http.common.HttpBinding;
 import org.apache.camel.spi.UriEndpoint;
 import org.apache.camel.util.URISupport;
 import org.apache.camel.util.UnsafeUriCharactersEncoder;
-import org.apache.commons.httpclient.HttpConnectionManager;
-import org.apache.commons.httpclient.params.HttpClientParams;
 
 /**
  * Represents a <a href="http://camel.apache.org/ghttp.html";>Google App Engine
@@ -62,12 +59,9 @@ public class GHttpEndpoint extends ServletEndpoint 
implements OutboundBindingSup
     private OutboundBinding<GHttpEndpoint, HTTPRequest, HTTPResponse> 
outboundBinding;
     private InboundBinding<GHttpEndpoint, HttpServletRequest, 
HttpServletResponse> inboundBinding;
     
-    public GHttpEndpoint(String endpointUri, ServletComponent component,
-            URI httpUri, HttpClientParams params,
-            HttpConnectionManager httpConnectionManager,
-            HttpClientConfigurer clientConfigurer) throws URISyntaxException {
+    public GHttpEndpoint(String endpointUri, ServletComponent component, URI 
httpUri) throws URISyntaxException {
         // set the endpoint uri with httpUri as we need to create http 
producer here
-        super(httpUri.toString(), component, httpUri, params, 
httpConnectionManager, clientConfigurer);
+        super(httpUri.toString(), component, httpUri);
         urlFetchService = URLFetchServiceFactory.getURLFetchService();
     }
 
@@ -129,12 +123,12 @@ public class GHttpEndpoint extends ServletEndpoint 
implements OutboundBindingSup
     }
 
     /**
-     * Proxies the {@link HttpBinding} returned by {@link super#getBinding()}
+     * Proxies the {@link org.apache.camel.http.common.HttpBinding} returned 
by {@link super#getBinding()}
      * with a dynamic proxy. The proxy's invocation handler further delegates 
to
      * {@link InboundBinding#readRequest(org.apache.camel.Endpoint, Exchange, 
Object)}
      * .
      * 
-     * @return proxied {@link HttpBinding}.
+     * @return proxied {@link org.apache.camel.http.common.HttpBinding}.
      */
     @Override
     public HttpBinding getBinding() {

http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskBinding.java
----------------------------------------------------------------------
diff --git 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskBinding.java
 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskBinding.java
index 1ca9acd..ba971d3 100644
--- 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskBinding.java
+++ 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskBinding.java
@@ -20,19 +20,16 @@ import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
 import com.google.appengine.api.taskqueue.TaskOptions;
-
 import org.apache.camel.Exchange;
 import org.apache.camel.Message;
 import org.apache.camel.component.gae.bind.InboundBinding;
 import org.apache.camel.component.gae.bind.OutboundBinding;
-import org.apache.camel.component.http.DefaultHttpBinding;
-import org.apache.camel.component.http.HttpMessage;
 import org.apache.camel.spi.HeaderFilterStrategy;
 
 /**
  * Binds the {@link TaskOptions} of the task queueing service to a Camel
  * {@link Exchange} for outbound communication. For inbound communication a
- * {@link HttpMessage} is bound to {@link Exchange}.
+ * {@link org.apache.camel.http.common.HttpMessage} is bound to {@link 
Exchange}.
  */
 public class GTaskBinding implements 
     OutboundBinding <GTaskEndpoint, TaskOptions, Void>,
@@ -98,7 +95,7 @@ public class GTaskBinding implements
      * @see GTaskBinding#GTASK_QUEUE_NAME
      * @see GTaskBinding#GTASK_TASK_NAME
      * @see GTaskBinding#GTASK_RETRY_COUNT
-     * @see DefaultHttpBinding#readRequest(HttpServletRequest, HttpMessage)
+     * @see 
org.apache.camel.http.common.DefaultHttpBinding#readRequest(HttpServletRequest, 
org.apache.camel.http.common.HttpMessage)
      */
     public Exchange readRequest(GTaskEndpoint endpoint, Exchange exchange, 
HttpServletRequest request) {
         readRequestHeaders(endpoint, exchange, request);

http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskComponent.java
----------------------------------------------------------------------
diff --git 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskComponent.java
 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskComponent.java
index d248d5d..da029e6 100644
--- 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskComponent.java
+++ 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskComponent.java
@@ -18,22 +18,17 @@ package org.apache.camel.component.gae.task;
 
 import java.net.URI;
 import java.util.Map;
-
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
 import com.google.appengine.api.taskqueue.QueueFactory;
 import com.google.appengine.api.taskqueue.TaskOptions;
-
 import org.apache.camel.Endpoint;
 import org.apache.camel.component.gae.bind.InboundBinding;
 import org.apache.camel.component.gae.bind.OutboundBinding;
-import org.apache.camel.component.http.HttpClientConfigurer;
-import org.apache.camel.component.http.HttpConsumer;
 import org.apache.camel.component.servlet.ServletComponent;
 import org.apache.camel.component.servlet.ServletEndpoint;
-import org.apache.commons.httpclient.HttpConnectionManager;
-import org.apache.commons.httpclient.params.HttpClientParams;
+import org.apache.camel.http.common.HttpConsumer;
 
 /**
  * The <a href="http://camel.apache.org/gtask.html";>Google App Engine Task
@@ -79,12 +74,8 @@ public class GTaskComponent extends ServletComponent {
     }
 
     @Override
-    protected ServletEndpoint createServletEndpoint(String endpointUri,
-            ServletComponent component, URI httpUri, HttpClientParams params,
-            HttpConnectionManager httpConnectionManager,
-            HttpClientConfigurer clientConfigurer) throws Exception {
-        return new GTaskEndpoint(endpointUri, component, httpUri, params,
-                httpConnectionManager, clientConfigurer);
+    protected ServletEndpoint createServletEndpoint(String endpointUri, 
ServletComponent component, URI httpUri) throws Exception {
+        return new GTaskEndpoint(endpointUri, component, httpUri);
     }
 
     @Override

http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskEndpoint.java
----------------------------------------------------------------------
diff --git 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskEndpoint.java
 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskEndpoint.java
index 371094e..e688d67 100644
--- 
a/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskEndpoint.java
+++ 
b/components/camel-gae/src/main/java/org/apache/camel/component/gae/task/GTaskEndpoint.java
@@ -30,16 +30,13 @@ import 
org.apache.camel.component.gae.bind.HttpBindingInvocationHandler;
 import org.apache.camel.component.gae.bind.InboundBinding;
 import org.apache.camel.component.gae.bind.OutboundBinding;
 import org.apache.camel.component.gae.bind.OutboundBindingSupport;
-import org.apache.camel.component.http.HttpBinding;
-import org.apache.camel.component.http.HttpClientConfigurer;
 import org.apache.camel.component.servlet.ServletComponent;
 import org.apache.camel.component.servlet.ServletEndpoint;
+import org.apache.camel.http.common.HttpBinding;
 import org.apache.camel.spi.Metadata;
 import org.apache.camel.spi.UriEndpoint;
 import org.apache.camel.spi.UriParam;
 import org.apache.camel.spi.UriPath;
-import org.apache.commons.httpclient.HttpConnectionManager;
-import org.apache.commons.httpclient.params.HttpClientParams;
 
 /**
  * Represents a <a href="http://camel.apache.org/gtask.html";>Google App Engine 
Task Queueing endpoint</a>.
@@ -61,11 +58,8 @@ public class GTaskEndpoint extends ServletEndpoint 
implements OutboundBindingSup
     @UriParam(label = "producer")
     private String outboundBindingRef;
 
-    public GTaskEndpoint(String endpointUri, ServletComponent component,
-            URI httpUri, HttpClientParams params,
-            HttpConnectionManager httpConnectionManager,
-            HttpClientConfigurer clientConfigurer) throws URISyntaxException {
-        super(endpointUri, component, httpUri, params, httpConnectionManager, 
clientConfigurer);
+    public GTaskEndpoint(String endpointUri, ServletComponent component, URI 
httpUri) throws URISyntaxException {
+        super(endpointUri, component, httpUri);
     }
 
     public OutboundBinding<GTaskEndpoint, TaskOptions, Void> 
getOutboundBinding() {

http://git-wip-us.apache.org/repos/asf/camel/blob/05019f6f/components/camel-gae/src/test/java/org/apache/camel/component/gae/task/GTaskBindingTest.java
----------------------------------------------------------------------
diff --git 
a/components/camel-gae/src/test/java/org/apache/camel/component/gae/task/GTaskBindingTest.java
 
b/components/camel-gae/src/test/java/org/apache/camel/component/gae/task/GTaskBindingTest.java
index 8cc42df..0154529 100644
--- 
a/components/camel-gae/src/test/java/org/apache/camel/component/gae/task/GTaskBindingTest.java
+++ 
b/components/camel-gae/src/test/java/org/apache/camel/component/gae/task/GTaskBindingTest.java
@@ -27,7 +27,7 @@ import 
com.google.appengine.api.taskqueue.dev.QueueStateInfo.TaskStateInfo;
 import com.google.appengine.tools.development.testing.LocalServiceTestHelper;
 import com.google.appengine.tools.development.testing.LocalTaskQueueTestConfig;
 import org.apache.camel.Exchange;
-import org.apache.camel.component.http.HttpMessage;
+import org.apache.camel.http.common.HttpMessage;
 import org.apache.camel.impl.DefaultExchange;
 import org.junit.After;
 import org.junit.Before;

Reply via email to