You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ts...@apache.org on 2020/01/30 00:24:44 UTC
[camel] branch master updated: CAMEL-14456: camel-master - RAW()
parameter value in delegated URI gets encoded
This is an automated email from the ASF dual-hosted git repository.
tsato pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/master by this push:
new a1487d1 CAMEL-14456: camel-master - RAW() parameter value in delegated URI gets encoded
a1487d1 is described below
commit a1487d12b9b9e6b57d98b14e334999b0c45dc7f5
Author: Tadayoshi Sato <sa...@gmail.com>
AuthorDate: Wed Jan 29 17:04:16 2020 +0900
CAMEL-14456: camel-master - RAW() parameter value in delegated URI gets encoded
---
.../camel/component/master/MasterComponent.java | 6 ++
.../component/master/EndpointUriEncodingTest.java | 103 +++++++++++++++++++++
2 files changed, 109 insertions(+)
diff --git a/components/camel-master/src/main/java/org/apache/camel/component/master/MasterComponent.java b/components/camel-master/src/main/java/org/apache/camel/component/master/MasterComponent.java
index 32838ad..8317823 100644
--- a/components/camel-master/src/main/java/org/apache/camel/component/master/MasterComponent.java
+++ b/components/camel-master/src/main/java/org/apache/camel/component/master/MasterComponent.java
@@ -78,6 +78,12 @@ public class MasterComponent extends DefaultComponent {
);
}
+ @Override
+ public boolean useRawUri() {
+ // disable URI encoding at master endpoint level to avoid encoding URI twice
+ return true;
+ }
+
public CamelClusterService getService() {
return service;
}
diff --git a/components/camel-master/src/test/java/org/apache/camel/component/master/EndpointUriEncodingTest.java b/components/camel-master/src/test/java/org/apache/camel/component/master/EndpointUriEncodingTest.java
new file mode 100644
index 0000000..1c46091
--- /dev/null
+++ b/components/camel-master/src/test/java/org/apache/camel/component/master/EndpointUriEncodingTest.java
@@ -0,0 +1,103 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.master;
+
+import java.util.Map;
+
+import org.apache.camel.Consumer;
+import org.apache.camel.Endpoint;
+import org.apache.camel.Exchange;
+import org.apache.camel.Processor;
+import org.apache.camel.Producer;
+import org.apache.camel.builder.RouteBuilder;
+import org.apache.camel.component.file.cluster.FileLockClusterService;
+import org.apache.camel.component.mock.MockEndpoint;
+import org.apache.camel.support.DefaultComponent;
+import org.apache.camel.support.DefaultConsumer;
+import org.apache.camel.support.DefaultEndpoint;
+import org.apache.camel.test.junit4.CamelTestSupport;
+import org.junit.Test;
+
+public class EndpointUriEncodingTest extends CamelTestSupport {
+
+ @Test
+ public void test() throws InterruptedException {
+ MockEndpoint mock = getMockEndpoint("mock:result");
+ mock.expectedMessageCount(1);
+ mock.message(0).header("foo").isEqualTo("hello} world");
+ mock.message(0).header("bar").isEqualTo("hello}+world");
+ mock.assertIsSatisfied();
+ }
+
+ @Override
+ protected RouteBuilder createRouteBuilder() throws Exception {
+ setupClusterService();
+ return new RouteBuilder() {
+ public void configure() {
+ context.addComponent("dummy", new DummyComponent());
+ from("master:test:dummy://path?foo=hello}+world&bar=RAW(hello}+world)")
+ .to("mock:result");
+ }
+ };
+ }
+
+ private void setupClusterService() throws Exception {
+ FileLockClusterService fileLockClusterService = new FileLockClusterService();
+ fileLockClusterService.setRoot(System.getProperty("java.io.tmpdir"));
+ context().addService(fileLockClusterService);
+ }
+
+ private class DummyComponent extends DefaultComponent {
+ @Override
+ protected Endpoint createEndpoint(String uri, String remaining, Map<String, Object> parameters) {
+ return new DefaultEndpoint(uri, this) {
+ private String foo;
+ private String bar;
+
+ public void setFoo(String foo) {
+ this.foo = foo;
+ }
+
+ public void setBar(String bar) {
+ this.bar = bar;
+ }
+
+ @Override
+ public Producer createProducer() {
+ return null;
+ }
+
+ @Override
+ public Consumer createConsumer(Processor processor) {
+ return new DefaultConsumer(this, processor) {
+ @Override
+ public void start() {
+ Exchange exchange = createExchange();
+ exchange.getMessage().setHeader("foo", foo);
+ exchange.getMessage().setHeader("bar", bar);
+ try {
+ getProcessor().process(exchange);
+ } catch (Exception e) {
+ }
+ }
+ };
+ }
+ };
+ }
+ }
+
+}