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 2019/08/22 06:15:31 UTC
[camel] 05/07: CAMEL-13870: Fast property configuration of Camel
endpoints. Work in progress.
This is an automated email from the ASF dual-hosted git repository.
davsclaus pushed a commit to branch CAMEL-13870
in repository https://gitbox.apache.org/repos/asf/camel.git
commit 287502df3d57c47d1131b2b0ac2bc9d668274786
Author: Claus Ibsen <cl...@gmail.com>
AuthorDate: Thu Aug 22 06:54:44 2019 +0200
CAMEL-13870: Fast property configuration of Camel endpoints. Work in progress.
---
.../camel/component/rest/swagger/RestSwaggerEndpoint.java | 12 +++++-------
.../main/java/org/apache/camel/support/DefaultEndpoint.java | 2 +-
2 files changed, 6 insertions(+), 8 deletions(-)
diff --git a/components/camel-rest-swagger/src/main/java/org/apache/camel/component/rest/swagger/RestSwaggerEndpoint.java b/components/camel-rest-swagger/src/main/java/org/apache/camel/component/rest/swagger/RestSwaggerEndpoint.java
index 6bcae93..7823e63 100644
--- a/components/camel-rest-swagger/src/main/java/org/apache/camel/component/rest/swagger/RestSwaggerEndpoint.java
+++ b/components/camel-rest-swagger/src/main/java/org/apache/camel/component/rest/swagger/RestSwaggerEndpoint.java
@@ -281,13 +281,11 @@ public final class RestSwaggerEndpoint extends DefaultEndpoint {
Map<String, Object> params = determineEndpointParameters(swagger, operation);
boolean hasHost = params.containsKey("host");
- setProperties(endpoint, params);
-
- // ensure rest configuration is available
-// final String componentName = determineComponentName();
-// if (componentName != null) {
-// RestConfiguration config = camelContext.getRestConfiguration(componentName, true);
-// }
+ if (endpoint instanceof DefaultEndpoint) {
+ // let the rest endpoint configure itself
+ DefaultEndpoint de = (DefaultEndpoint) endpoint;
+ de.setProperties(endpoint, params);
+ }
// if there is a host then we should use this hardcoded host instead of any Header that may have an existing
// Host header from some other HTTP input, and if so then lets remove it
diff --git a/core/camel-support/src/main/java/org/apache/camel/support/DefaultEndpoint.java b/core/camel-support/src/main/java/org/apache/camel/support/DefaultEndpoint.java
index de7dd80..14c5900 100644
--- a/core/camel-support/src/main/java/org/apache/camel/support/DefaultEndpoint.java
+++ b/core/camel-support/src/main/java/org/apache/camel/support/DefaultEndpoint.java
@@ -409,7 +409,7 @@ public abstract class DefaultEndpoint extends ServiceSupport implements Endpoint
* @param bean the bean
* @param parameters properties to set
*/
- protected void setProperties(Object bean, Map<String, Object> parameters) throws Exception {
+ public void setProperties(Object bean, Map<String, Object> parameters) throws Exception {
if (basicPropertyBinding) {
// use basic binding
PropertyBindingSupport.build()