You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by nf...@apache.org on 2017/03/07 13:57:26 UTC
[1/2] camel git commit: CAMEL-10951: rebuild module starters
Repository: camel
Updated Branches:
refs/heads/master 351ee03be -> cace0ee7e
CAMEL-10951: rebuild module starters
Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/cace0ee7
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/cace0ee7
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/cace0ee7
Branch: refs/heads/master
Commit: cace0ee7e03cf5a8c14b1ca7624c8210afcb0ffe
Parents: c441e1f
Author: Nicola Ferraro <ni...@gmail.com>
Authored: Tue Mar 7 14:54:23 2017 +0100
Committer: Nicola Ferraro <ni...@gmail.com>
Committed: Tue Mar 7 14:57:13 2017 +0100
----------------------------------------------------------------------
.../component/box/springboot/BoxComponentConfiguration.java | 2 +-
.../ganglia/springboot/GangliaComponentConfiguration.java | 2 +-
.../component/mail/springboot/MailComponentConfiguration.java | 6 +++---
.../netty/springboot/NettyComponentConfiguration.java | 2 +-
.../netty4/springboot/NettyComponentConfiguration.java | 2 +-
.../component/ssh/springboot/SshComponentConfiguration.java | 2 +-
.../springboot/XmlSignatureComponentConfiguration.java | 2 +-
.../spring-boot-dm/camel-spring-boot-dependencies/pom.xml | 2 +-
8 files changed, 10 insertions(+), 10 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/camel/blob/cace0ee7/platforms/spring-boot/components-starter/camel-box-starter/src/main/java/org/apache/camel/component/box/springboot/BoxComponentConfiguration.java
----------------------------------------------------------------------
diff --git a/platforms/spring-boot/components-starter/camel-box-starter/src/main/java/org/apache/camel/component/box/springboot/BoxComponentConfiguration.java b/platforms/spring-boot/components-starter/camel-box-starter/src/main/java/org/apache/camel/component/box/springboot/BoxComponentConfiguration.java
index d60ba98..ff84a54 100644
--- a/platforms/spring-boot/components-starter/camel-box-starter/src/main/java/org/apache/camel/component/box/springboot/BoxComponentConfiguration.java
+++ b/platforms/spring-boot/components-starter/camel-box-starter/src/main/java/org/apache/camel/component/box/springboot/BoxComponentConfiguration.java
@@ -124,7 +124,7 @@ public class BoxComponentConfiguration {
* @param authenticationTypethe
* authenticationType to set
*/
- private String authenticationType = "APP_USER_AUTHENTICATION";
+ private String authenticationType = org.apache.camel.component.box.BoxConfiguration.APP_USER_AUTHENTICATION;
/**
* Box application client ID
*
http://git-wip-us.apache.org/repos/asf/camel/blob/cace0ee7/platforms/spring-boot/components-starter/camel-ganglia-starter/src/main/java/org/apache/camel/component/ganglia/springboot/GangliaComponentConfiguration.java
----------------------------------------------------------------------
diff --git a/platforms/spring-boot/components-starter/camel-ganglia-starter/src/main/java/org/apache/camel/component/ganglia/springboot/GangliaComponentConfiguration.java b/platforms/spring-boot/components-starter/camel-ganglia-starter/src/main/java/org/apache/camel/component/ganglia/springboot/GangliaComponentConfiguration.java
index 5872e71..e0b7b65 100644
--- a/platforms/spring-boot/components-starter/camel-ganglia-starter/src/main/java/org/apache/camel/component/ganglia/springboot/GangliaComponentConfiguration.java
+++ b/platforms/spring-boot/components-starter/camel-ganglia-starter/src/main/java/org/apache/camel/component/ganglia/springboot/GangliaComponentConfiguration.java
@@ -65,7 +65,7 @@ public class GangliaComponentConfiguration {
/**
* Host name for Ganglia server
*/
- private String host = "DEFAULT_DESTINATION";
+ private String host = org.apache.camel.component.ganglia.GangliaConfiguration.DEFAULT_DESTINATION;
/**
* Port for Ganglia server
*/
http://git-wip-us.apache.org/repos/asf/camel/blob/cace0ee7/platforms/spring-boot/components-starter/camel-mail-starter/src/main/java/org/apache/camel/component/mail/springboot/MailComponentConfiguration.java
----------------------------------------------------------------------
diff --git a/platforms/spring-boot/components-starter/camel-mail-starter/src/main/java/org/apache/camel/component/mail/springboot/MailComponentConfiguration.java b/platforms/spring-boot/components-starter/camel-mail-starter/src/main/java/org/apache/camel/component/mail/springboot/MailComponentConfiguration.java
index 6937ee2..c0a5097 100644
--- a/platforms/spring-boot/components-starter/camel-mail-starter/src/main/java/org/apache/camel/component/mail/springboot/MailComponentConfiguration.java
+++ b/platforms/spring-boot/components-starter/camel-mail-starter/src/main/java/org/apache/camel/component/mail/springboot/MailComponentConfiguration.java
@@ -131,7 +131,7 @@ public class MailComponentConfiguration {
/**
* The from email address
*/
- private String from = "MailConstants.MAIL_DEFAULT_FROM";
+ private String from = org.apache.camel.component.mail.MailConstants.MAIL_DEFAULT_FROM;
/**
* Deletes the messages after they have been processed. This is done by
* setting the DELETED flag on the mail message. If false, the SEEN flag
@@ -152,7 +152,7 @@ public class MailComponentConfiguration {
/**
* The folder to poll.
*/
- private String folderName = "MailConstants.MAIL_DEFAULT_FOLDER";
+ private String folderName = org.apache.camel.component.mail.MailConstants.MAIL_DEFAULT_FOLDER;
/**
* Option to let Camel ignore unsupported charset in the local JVM when
* sending mails. If the charset is unsupported then charset=XXX (where
@@ -218,7 +218,7 @@ public class MailComponentConfiguration {
* email clients, set the alternative mail body with this key as a
* header.
*/
- private String alternativeBodyHeader = "MailConstants.MAIL_ALTERNATIVE_BODY";
+ private String alternativeBodyHeader = org.apache.camel.component.mail.MailConstants.MAIL_ALTERNATIVE_BODY;
/**
* Whether to use disposition inline or attachment.
*/
http://git-wip-us.apache.org/repos/asf/camel/blob/cace0ee7/platforms/spring-boot/components-starter/camel-netty-starter/src/main/java/org/apache/camel/component/netty/springboot/NettyComponentConfiguration.java
----------------------------------------------------------------------
diff --git a/platforms/spring-boot/components-starter/camel-netty-starter/src/main/java/org/apache/camel/component/netty/springboot/NettyComponentConfiguration.java b/platforms/spring-boot/components-starter/camel-netty-starter/src/main/java/org/apache/camel/component/netty/springboot/NettyComponentConfiguration.java
index 3ff6dd5..1f6c05e 100644
--- a/platforms/spring-boot/components-starter/camel-netty-starter/src/main/java/org/apache/camel/component/netty/springboot/NettyComponentConfiguration.java
+++ b/platforms/spring-boot/components-starter/camel-netty-starter/src/main/java/org/apache/camel/component/netty/springboot/NettyComponentConfiguration.java
@@ -442,7 +442,7 @@ public class NettyComponentConfiguration {
/**
* Which protocols to enable when using SSL
*/
- private String enabledProtocols = "DEFAULT_ENABLED_PROTOCOLS";
+ private String enabledProtocols = org.apache.camel.component.netty.NettyConfiguration.DEFAULT_ENABLED_PROTOCOLS;
public Long getRequestTimeout() {
return requestTimeout;
http://git-wip-us.apache.org/repos/asf/camel/blob/cace0ee7/platforms/spring-boot/components-starter/camel-netty4-starter/src/main/java/org/apache/camel/component/netty4/springboot/NettyComponentConfiguration.java
----------------------------------------------------------------------
diff --git a/platforms/spring-boot/components-starter/camel-netty4-starter/src/main/java/org/apache/camel/component/netty4/springboot/NettyComponentConfiguration.java b/platforms/spring-boot/components-starter/camel-netty4-starter/src/main/java/org/apache/camel/component/netty4/springboot/NettyComponentConfiguration.java
index 0086e44..8981d12 100644
--- a/platforms/spring-boot/components-starter/camel-netty4-starter/src/main/java/org/apache/camel/component/netty4/springboot/NettyComponentConfiguration.java
+++ b/platforms/spring-boot/components-starter/camel-netty4-starter/src/main/java/org/apache/camel/component/netty4/springboot/NettyComponentConfiguration.java
@@ -481,7 +481,7 @@ public class NettyComponentConfiguration {
/**
* Which protocols to enable when using SSL
*/
- private String enabledProtocols = "DEFAULT_ENABLED_PROTOCOLS";
+ private String enabledProtocols = org.apache.camel.component.netty4.NettyConfiguration.DEFAULT_ENABLED_PROTOCOLS;
private Boolean reconnect;
private Integer reconnectInterval;
http://git-wip-us.apache.org/repos/asf/camel/blob/cace0ee7/platforms/spring-boot/components-starter/camel-ssh-starter/src/main/java/org/apache/camel/component/ssh/springboot/SshComponentConfiguration.java
----------------------------------------------------------------------
diff --git a/platforms/spring-boot/components-starter/camel-ssh-starter/src/main/java/org/apache/camel/component/ssh/springboot/SshComponentConfiguration.java b/platforms/spring-boot/components-starter/camel-ssh-starter/src/main/java/org/apache/camel/component/ssh/springboot/SshComponentConfiguration.java
index 3d59c0e..03f4d82 100644
--- a/platforms/spring-boot/components-starter/camel-ssh-starter/src/main/java/org/apache/camel/component/ssh/springboot/SshComponentConfiguration.java
+++ b/platforms/spring-boot/components-starter/camel-ssh-starter/src/main/java/org/apache/camel/component/ssh/springboot/SshComponentConfiguration.java
@@ -257,7 +257,7 @@ public class SshComponentConfiguration {
* authentication.
* @see KeyPairProvider
*/
- private String keyType = "KeyPairProvider.SSH_RSA";
+ private String keyType = org.apache.sshd.common.KeyPairProvider.SSH_RSA;
/**
* Sets the timeout in milliseconds to wait in establishing the remote
* SSH server connection. Defaults to 30000 milliseconds.
http://git-wip-us.apache.org/repos/asf/camel/blob/cace0ee7/platforms/spring-boot/components-starter/camel-xmlsecurity-starter/src/main/java/org/apache/camel/component/xmlsecurity/springboot/XmlSignatureComponentConfiguration.java
----------------------------------------------------------------------
diff --git a/platforms/spring-boot/components-starter/camel-xmlsecurity-starter/src/main/java/org/apache/camel/component/xmlsecurity/springboot/XmlSignatureComponentConfiguration.java b/platforms/spring-boot/components-starter/camel-xmlsecurity-starter/src/main/java/org/apache/camel/component/xmlsecurity/springboot/XmlSignatureComponentConfiguration.java
index 4d6be26..e6e1f04 100644
--- a/platforms/spring-boot/components-starter/camel-xmlsecurity-starter/src/main/java/org/apache/camel/component/xmlsecurity/springboot/XmlSignatureComponentConfiguration.java
+++ b/platforms/spring-boot/components-starter/camel-xmlsecurity-starter/src/main/java/org/apache/camel/component/xmlsecurity/springboot/XmlSignatureComponentConfiguration.java
@@ -644,7 +644,7 @@ public class XmlSignatureComponentConfiguration {
* {@link #setOutputNodeSearch(Object)} . The supported default search
* types you can find in {@link DefaultXmlSignature2Message} .
*/
- private String outputNodeSearchType = "DefaultXmlSignature2Message.OUTPUT_NODE_SEARCH_TYPE_DEFAULT";
+ private String outputNodeSearchType = org.apache.camel.component.xmlsecurity.api.DefaultXmlSignature2Message.OUTPUT_NODE_SEARCH_TYPE_DEFAULT;
/**
* Indicator whether the XML signature elements (elements with local
* name "Signature" and namesapce
http://git-wip-us.apache.org/repos/asf/camel/blob/cace0ee7/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml
----------------------------------------------------------------------
diff --git a/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml b/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml
index 38de9fb..cb17ae8 100644
--- a/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml
+++ b/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml
@@ -157,7 +157,7 @@
<dependency>
<groupId>net.sf.saxon</groupId>
<artifactId>Saxon-HE</artifactId>
- <version>9.7.0-14</version>
+ <version>9.7.0-15</version>
</dependency>
<dependency>
<groupId>ognl</groupId>
[2/2] camel git commit: CAMEL-10951: fix constant literals in
spring-boot configuration classes
Posted by nf...@apache.org.
CAMEL-10951: fix constant literals in spring-boot configuration classes
Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/c441e1f2
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/c441e1f2
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/c441e1f2
Branch: refs/heads/master
Commit: c441e1f220351ae2aeb0ff7e5edd98e971304eec
Parents: 351ee03
Author: Nicola Ferraro <ni...@gmail.com>
Authored: Tue Mar 7 14:53:54 2017 +0100
Committer: Nicola Ferraro <ni...@gmail.com>
Committed: Tue Mar 7 14:57:13 2017 +0100
----------------------------------------------------------------------
.../SpringBootAutoConfigurationMojo.java | 69 ++++++++++++++++++--
1 file changed, 65 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/camel/blob/c441e1f2/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/SpringBootAutoConfigurationMojo.java
----------------------------------------------------------------------
diff --git a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/SpringBootAutoConfigurationMojo.java b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/SpringBootAutoConfigurationMojo.java
index e54a49e..cd24d77 100644
--- a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/SpringBootAutoConfigurationMojo.java
+++ b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/SpringBootAutoConfigurationMojo.java
@@ -39,11 +39,13 @@ import java.util.Locale;
import java.util.Map;
import java.util.Set;
import java.util.TreeSet;
+import java.util.regex.Matcher;
import java.util.regex.Pattern;
import java.util.stream.Collectors;
import com.google.gson.Gson;
import com.google.gson.GsonBuilder;
+
import org.apache.camel.maven.packaging.model.ComponentModel;
import org.apache.camel.maven.packaging.model.ComponentOptionModel;
import org.apache.camel.maven.packaging.model.DataFormatModel;
@@ -132,6 +134,8 @@ public class SpringBootAutoConfigurationMojo extends AbstractMojo {
PRIMITIVEMAP.put("float", "java.lang.Float");
}
+ private static final List<String> JAVA_LANG_TYPES = Arrays.asList("Boolean", "Byte", "Character", "Class", "Double", "Float", "Integer", "Long", "Object", "Short", "String");
+
private static final String[] IGNORE_MODULES = {/* Non-standard -> */ "camel-grape", "camel-connector"};
/**
@@ -464,16 +468,20 @@ public class SpringBootAutoConfigurationMojo extends AbstractMojo {
}
String defaultValue = null;
+ String defaultValueLiteral = null;
if (sourceProp.hasAnnotation(UriParam.class)) {
defaultValue = sourceProp.getAnnotation(UriParam.class).getStringValue("defaultValue");
+ defaultValueLiteral = sourceProp.getAnnotation(UriParam.class).getLiteralValue("defaultValue");
} else if (sourceProp.hasAnnotation(UriPath.class)) {
defaultValue = sourceProp.getAnnotation(UriPath.class).getStringValue("defaultValue");
+ defaultValueLiteral = sourceProp.getAnnotation(UriPath.class).getLiteralValue("defaultValue");
}
+
+ defaultValueLiteral = makeFQ(nestedType, defaultValueLiteral);
+
if (!Strings.isBlank(defaultValue)) {
- if ("java.lang.String".equals(optionType)) {
- prop.getField().setStringInitializer(defaultValue);
- } else if ("integer".equals(optionType) || "boolean".equals(optionType)) {
- prop.getField().setLiteralInitializer(defaultValue);
+ if ("integer".equals(optionType) || "boolean".equals(optionType) || "java.lang.String".equals(optionType)) {
+ prop.getField().setLiteralInitializer(defaultValueLiteral);
} else if (anEnum) {
String enumShortName = optionClass.getSimpleName();
prop.getField().setLiteralInitializer(enumShortName + "." + defaultValue);
@@ -490,6 +498,59 @@ public class SpringBootAutoConfigurationMojo extends AbstractMojo {
writeSourceIfChanged(javaClass, fileName);
}
+ private String makeFQ(JavaClassSource source, String literal) {
+ if (literal == null) {
+ return null;
+ }
+
+ if (Pattern.matches("[A-Z][A-Z0-9_]*", literal)) {
+ return source.getQualifiedName() + "." + literal;
+ }
+
+ Map<String, String> fq = new HashMap<>();
+ List<String> classes = extractClasses(literal);
+ for (String cl : classes) {
+
+ boolean found = false;
+ for (Import im : source.getImports()) {
+ if (cl.equals(im.getSimpleName())) {
+ fq.put(cl, im.getQualifiedName());
+ found = true;
+ break;
+ }
+ }
+
+ if (!found) {
+ // if it's not a java.lang object, then it's in the same package
+ if (!JAVA_LANG_TYPES.contains(cl)) {
+ fq.put(cl, source.getPackage() + "." + cl);
+ }
+ }
+ }
+
+ if (fq.size() > 0) {
+ String res = literal;
+ for (Map.Entry<String, String> fqn : fq.entrySet()) {
+ res = res.replace(fqn.getKey(), fqn.getValue());
+ }
+ return res;
+ }
+ return literal;
+ }
+
+ private List<String> extractClasses(String literal) {
+ if (literal.startsWith("\"") && literal.endsWith("\"")) {
+ return Collections.emptyList();
+ }
+ List<String> classes = new LinkedList<>();
+ Pattern regex = Pattern.compile("[^A-Za-z0-9_.]*([A-Z][A-Za-z0-9]*)[.][A-Za-z0-9_.-]+");
+ Matcher m = regex.matcher(literal);
+ while (m.find()) {
+ classes.add(m.group(1));
+ }
+ return classes;
+ }
+
// resolved property type name and property source, Roaster doesn't resolve inner classes correctly
private class ResolvedProperty {
private String propertyType;