You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@camel.apache.org by kuro <96...@gmail.com> on 2014/04/11 12:39:00 UTC
Patch - camel-jetty
Hi
I solved the issue of
http://servicemix.396122.n5.nabble.com/HTTP-Date-headers-for-Jetty-endpoints-td5718803.html
Example:
jetty:http://0.0.0.0/?sendDateHeader=true
diff --git
a/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java
b/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java
index 670a4a5..adc504f 100644
--- a/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java
+++ b/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java
@@ -299,6 +299,8 @@ public class JettyHttpComponent extends HttpComponent {
// just set if we need sendServerVersion, the default value
is true
server.setSendServerVersion(endpoint.isSendServerVersion());
server.addConnector(connector);
+
+ server.setSendDateHeader(endpoint.isSendDateHeader());
connectorRef = new ConnectorRef(server, connector,
createServletForConnector(server, connector, endpoint.getHandlers(),
endpoint));
// must enable session before we start
diff --git
a/src/main/java/org/apache/camel/component/jetty/JettyHttpEndpoint.java
b/src/main/java/org/apache/camel/component/jetty/JettyHttpEndpoint.java
index 089e159..e1e64f7 100644
--- a/src/main/java/org/apache/camel/component/jetty/JettyHttpEndpoint.java
+++ b/src/main/java/org/apache/camel/component/jetty/JettyHttpEndpoint.java
@@ -49,6 +49,7 @@ public class JettyHttpEndpoint extends HttpEndpoint {
private boolean enableJmx;
private boolean enableMultipartFilter;
private boolean sendServerVersion = true;
+ private boolean sendDateHeader = false;
private Filter multipartFilter;
private List<Filter> filters;
private Long continuationTimeout;
@@ -176,6 +177,14 @@ public class JettyHttpEndpoint extends HttpEndpoint {
this.sendServerVersion = sendServerVersion;
}
+ public boolean isSendDateHeader() {
+ return sendDateHeader;
+ }
+
+ public void setSendDateHeader(boolean sendDateHeader) {
+ this.sendDateHeader = sendDateHeader;
+ }
+
public boolean isEnableMultipartFilter() {
return enableMultipartFilter;
}
--
View this message in context: http://camel.465427.n5.nabble.com/Patch-camel-jetty-tp5750112.html
Sent from the Camel - Users mailing list archive at Nabble.com.
Re: Patch - camel-jetty
Posted by Willem Jiang <wi...@gmail.com>.
I will take care of it.
--
Willem Jiang
Red Hat, Inc.
Web: http://www.redhat.com
Blog: http://willemjiang.blogspot.com (English)
http://jnn.iteye.com (Chinese)
Twitter: willemjiang
Weibo: 姜宁willem
On April 12, 2014 at 3:03:12 PM, kuro (96259d@gmail.com) wrote:
> https://issues.apache.org/jira/browse/CAMEL-7360
>
>
>
>
> --
> View this message in context: http://camel.465427.n5.nabble.com/Patch-camel-jetty-tp5750112p5750140.html
> Sent from the Camel - Users mailing list archive at Nabble.com.
>
Re: Patch - camel-jetty
Posted by kuro <96...@gmail.com>.
https://issues.apache.org/jira/browse/CAMEL-7360
--
View this message in context: http://camel.465427.n5.nabble.com/Patch-camel-jetty-tp5750112p5750140.html
Sent from the Camel - Users mailing list archive at Nabble.com.
Re: Patch - camel-jetty
Posted by Claus Ibsen <cl...@gmail.com>.
Hi
Sounds good. Do you mind create a JIRA ticket and attach your patch as
a file to the JIRA ticket?
Link to issue tracker here
http://camel.apache.org/support
On Fri, Apr 11, 2014 at 12:39 PM, kuro <96...@gmail.com> wrote:
> Hi
>
> I solved the issue of
> http://servicemix.396122.n5.nabble.com/HTTP-Date-headers-for-Jetty-endpoints-td5718803.html
>
> Example:
> jetty:http://0.0.0.0/?sendDateHeader=true
>
>
> diff --git
> a/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java
> b/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java
> index 670a4a5..adc504f 100644
> --- a/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java
> +++ b/src/main/java/org/apache/camel/component/jetty/JettyHttpComponent.java
> @@ -299,6 +299,8 @@ public class JettyHttpComponent extends HttpComponent {
> // just set if we need sendServerVersion, the default value
> is true
>
> server.setSendServerVersion(endpoint.isSendServerVersion());
> server.addConnector(connector);
> +
> + server.setSendDateHeader(endpoint.isSendDateHeader());
>
> connectorRef = new ConnectorRef(server, connector,
> createServletForConnector(server, connector, endpoint.getHandlers(),
> endpoint));
> // must enable session before we start
> diff --git
> a/src/main/java/org/apache/camel/component/jetty/JettyHttpEndpoint.java
> b/src/main/java/org/apache/camel/component/jetty/JettyHttpEndpoint.java
> index 089e159..e1e64f7 100644
> --- a/src/main/java/org/apache/camel/component/jetty/JettyHttpEndpoint.java
> +++ b/src/main/java/org/apache/camel/component/jetty/JettyHttpEndpoint.java
> @@ -49,6 +49,7 @@ public class JettyHttpEndpoint extends HttpEndpoint {
> private boolean enableJmx;
> private boolean enableMultipartFilter;
> private boolean sendServerVersion = true;
> + private boolean sendDateHeader = false;
> private Filter multipartFilter;
> private List<Filter> filters;
> private Long continuationTimeout;
> @@ -176,6 +177,14 @@ public class JettyHttpEndpoint extends HttpEndpoint {
> this.sendServerVersion = sendServerVersion;
> }
>
> + public boolean isSendDateHeader() {
> + return sendDateHeader;
> + }
> +
> + public void setSendDateHeader(boolean sendDateHeader) {
> + this.sendDateHeader = sendDateHeader;
> + }
> +
> public boolean isEnableMultipartFilter() {
> return enableMultipartFilter;
> }
>
>
>
> --
> View this message in context: http://camel.465427.n5.nabble.com/Patch-camel-jetty-tp5750112.html
> Sent from the Camel - Users mailing list archive at Nabble.com.
--
Claus Ibsen
-----------------
Red Hat, Inc.
Email: cibsen@redhat.com
Twitter: davsclaus
Blog: http://davsclaus.com
Author of Camel in Action: http://www.manning.com/ibsen
hawtio: http://hawt.io/
fabric8: http://fabric8.io/