You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ac...@apache.org on 2022/11/11 11:03:40 UTC
[camel] 01/04: Netty-Http component: header parameters cannot start with a whitespace
This is an automated email from the ASF dual-hosted git repository.
acosentino pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
commit 7e9a26cd4a4d41f1c0ae1eafa6a42a0cd57848ab
Author: Andrea Cosentino <an...@gmail.com>
AuthorDate: Fri Nov 11 11:55:03 2022 +0100
Netty-Http component: header parameters cannot start with a whitespace
Signed-off-by: Andrea Cosentino <an...@gmail.com>
---
.../component/netty/http/NettyHttpBridgeEncodedPathTest.java | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/components/camel-netty-http/src/test/java/org/apache/camel/component/netty/http/NettyHttpBridgeEncodedPathTest.java b/components/camel-netty-http/src/test/java/org/apache/camel/component/netty/http/NettyHttpBridgeEncodedPathTest.java
index 04004081431..1131fb26fdc 100644
--- a/components/camel-netty-http/src/test/java/org/apache/camel/component/netty/http/NettyHttpBridgeEncodedPathTest.java
+++ b/components/camel-netty-http/src/test/java/org/apache/camel/component/netty/http/NettyHttpBridgeEncodedPathTest.java
@@ -45,9 +45,9 @@ public class NettyHttpBridgeEncodedPathTest extends BaseNettyTest {
@Test
public void testEncodedQuery() {
- String response = template.requestBody("http://localhost:" + port2 + "/nettyTestRouteA?param1=%2B447777111222", null,
+ String response = template.requestBody("http://localhost:" + port2 + "/nettyTestRouteA?param1=44777%2B7111222", null,
String.class);
- assertEquals("param1=+447777111222", response, "Get a wrong response");
+ assertEquals("param1=44777+7111222", response, "Get a wrong response");
}
@Test
@@ -80,19 +80,21 @@ public class NettyHttpBridgeEncodedPathTest extends BaseNettyTest {
// %2B becomes decoded to a space
Object s = exchange.getIn().getHeader("param1");
// can be either + or %2B
- assertTrue(s.equals(" 447777111222") || s.equals("%20447777111222") || s.equals("+447777111222")
- || s.equals("%2B447777111222"));
+ assertTrue(s.equals("44777 7111222") || s.equals("44777%207111222") || s.equals("44777+7111222")
+ || s.equals("44777%2B7111222"));
// send back the query
exchange.getMessage().setBody(exchange.getIn().getHeader(Exchange.HTTP_QUERY));
};
from("netty-http:http://localhost:" + port2 + "/nettyTestRouteA?matchOnUriPrefix=true")
+ .log("${body} ${headers}")
.log("Using NettyTestRouteA route: CamelHttpPath=[${header.CamelHttpPath}], CamelHttpUri=[${header.CamelHttpUri}]")
.to("netty-http:http://localhost:" + port1
+ "/nettyTestRouteB?throwExceptionOnFailure=false&bridgeEndpoint=true");
from("netty-http:http://localhost:" + port1 + "/nettyTestRouteB?matchOnUriPrefix=true")
+ .log("${body} ${headers}")
.log("Using NettyTestRouteB route: CamelHttpPath=[${header.CamelHttpPath}], CamelHttpUri=[${header.CamelHttpUri}]")
.process(serviceProc);