camel git commit: CAMEL-10228: camel-sql - Preserver attachments

2016-08-20 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 24c92d464 -> 2d4194852


CAMEL-10228: camel-sql - Preserver attachments


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/2d419485
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/2d419485
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/2d419485

Branch: refs/heads/master
Commit: 2d4194852f1ca11c5376d7c6843ff4962060198d
Parents: 24c92d4
Author: Arno Noordover 
Authored: Sat Aug 20 22:20:04 2016 +0200
Committer: Arno Noordover 
Committed: Sat Aug 20 22:20:04 2016 +0200

--
 .../apache/camel/component/sql/SqlProducer.java |  4 +
 .../sql/SqlProducerOutputAttachment.java| 96 
 2 files changed, 100 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/2d419485/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlProducer.java
--
diff --git 
a/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlProducer.java
 
b/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlProducer.java
index f369234..2467335 100644
--- 
a/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlProducer.java
+++ 
b/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlProducer.java
@@ -162,6 +162,7 @@ public class SqlProducer extends DefaultProducer {
 if (isResultSet) {
 // preserve headers first, so we can override the 
SQL_ROW_COUNT header
 
exchange.getOut().getHeaders().putAll(exchange.getIn().getHeaders());
+
exchange.getOut().getAttachments().putAll(exchange.getIn().getAttachments());
 
 rs = ps.getResultSet();
 SqlOutputType outputType = 
getEndpoint().getOutputType();
@@ -211,6 +212,7 @@ public class SqlProducer extends DefaultProducer {
 // if no OUT message yet then create one and propagate 
headers
 if (!exchange.hasOut()) {
 
exchange.getOut().getHeaders().putAll(exchange.getIn().getHeaders());
+
exchange.getOut().getAttachments().putAll(exchange.getIn().getAttachments());
 }
 
 if (isResultSet) {
@@ -282,6 +284,8 @@ public class SqlProducer extends DefaultProducer {
 ResultSetIterator iterator = 
getEndpoint().queryForStreamList(con, ps, rs);
 //pass through all headers
 
exchange.getOut().getHeaders().putAll(exchange.getIn().getHeaders());
+
exchange.getOut().getAttachments().putAll(exchange.getIn().getAttachments());
+
 if (getEndpoint().isNoop()) {
 exchange.getOut().setBody(exchange.getIn().getBody());
 } else if (getEndpoint().getOutputHeader() != null) {

http://git-wip-us.apache.org/repos/asf/camel/blob/2d419485/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlProducerOutputAttachment.java
--
diff --git 
a/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlProducerOutputAttachment.java
 
b/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlProducerOutputAttachment.java
new file mode 100644
index 000..5b128fa
--- /dev/null
+++ 
b/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlProducerOutputAttachment.java
@@ -0,0 +1,96 @@
+/**
+ * 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.sql;
+
+import java.net.MalformedURLException;
+import java.net.URL;
+import javax.activation.DataHandler;
+
+import org.apache.camel.Exchange;
+import org.apache.camel.builder.RouteBuilder;
+import org.apache.camel.component.mock.MockEndpoint;
+import org.apache.camel.test.junit4.CamelTestSupport;
+import org.junit.After;
+import o

camel git commit: CAMEL-10090: execute spock test from maven

2016-08-13 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master db1f8eae1 -> a9d34dc87


CAMEL-10090: execute spock test from maven


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/a9d34dc8
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/a9d34dc8
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/a9d34dc8

Branch: refs/heads/master
Commit: a9d34dc87003acc05f07c7dc51c8760f07f45de1
Parents: db1f8ea
Author: Arno Noordover 
Authored: Sat Aug 13 20:57:09 2016 +0200
Committer: Arno Noordover 
Committed: Sat Aug 13 20:57:09 2016 +0200

--
 .../camel-salesforce-component/pom.xml  |  4 +-
 .../api/utils/DateTimeUtilsTest.groovy  | 56 
 .../api/utils/DateTimeUtilsTest.groovy  | 56 
 3 files changed, 58 insertions(+), 58 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/a9d34dc8/components/camel-salesforce/camel-salesforce-component/pom.xml
--
diff --git a/components/camel-salesforce/camel-salesforce-component/pom.xml 
b/components/camel-salesforce/camel-salesforce-component/pom.xml
index 7ce2548..4f14987 100644
--- a/components/camel-salesforce/camel-salesforce-component/pom.xml
+++ b/components/camel-salesforce/camel-salesforce-component/pom.xml
@@ -170,11 +170,11 @@
   
 org.codehaus.gmavenplus
 gmavenplus-plugin
-1.4
+1.5
 
   
 
-  compile
+  addTestSources
   testCompile
 
   

http://git-wip-us.apache.org/repos/asf/camel/blob/a9d34dc8/components/camel-salesforce/camel-salesforce-component/src/test/groovy/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/test/groovy/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
 
b/components/camel-salesforce/camel-salesforce-component/src/test/groovy/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
new file mode 100644
index 000..8452fe0
--- /dev/null
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/test/groovy/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
@@ -0,0 +1,56 @@
+/**
+ * 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.salesforce.api.utils
+
+import spock.lang.Ignore
+import spock.lang.Specification
+import spock.lang.Unroll
+
+import java.time.ZoneId
+import java.time.ZonedDateTime
+
+class DateTimeUtilsTest extends Specification {
+@Unroll
+def "Date #zonedDateTime should render as #result"() {
+expect:
+DateTimeUtils.formatDateTime(zonedDateTime) == result
+
+where:
+zonedDateTime  
|| result
+ZonedDateTime.of(1991, 12, 10, 12, 13, 14, 700, 
ZoneId.of("UTC+01:00:21")) || "1991-12-10T12:13:14.007+01:00"
+ZonedDateTime.of(1991, 12, 10, 12, 13, 14, 700, ZoneId.of("UTC"))  
|| "1991-12-10T12:13:14.007Z"
+ZonedDateTime.of(1700, 1, 1, 1, 13, 14, 700, 
ZoneId.of("UTC+00:19:21"))|| "1700-01-01T01:13:14.007+00:19"
+ZonedDateTime.of(1700, 2, 3, 2, 13, 14, 700, ZoneId.of("UTC")) 
|| "1700-02-03T02:13:14.007Z"
+}
+
+@Unroll
+def "Date #dateAsString should parse to #result"() {
+expect:
+DateTimeUtils.parseDateTime(dateAsString) == result
+
+where:
+dateAsString|| result
+"1991-12-10T12:13:14.007+01:00" || ZonedDateTime.of(1991, 12, 10, 12, 
13, 14, 700, ZoneId.of("+01:00"))
+"1991-12-10T12:13:14+00:00" || ZonedDateTime.of(1991, 12, 10, 12, 
13, 14, 0, ZoneId.of("Z"))
+"1991-12-10T12:13:14.000+00:00" || ZonedDateTime.of(1991, 12, 10, 12, 
13, 14, 0, ZoneId.of("Z"))
+"1991-12-10T12:13:

camel git commit: CAMEL-10090: changed according to test

2016-08-11 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 55a9eb80e -> 753100b4c


CAMEL-10090: changed according to test


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/753100b4
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/753100b4
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/753100b4

Branch: refs/heads/master
Commit: 753100b4c20f07df67977ec5f02c1120a7e08919
Parents: 55a9eb8
Author: Arno Noordover 
Authored: Thu Aug 11 21:58:50 2016 +0200
Committer: Arno Noordover 
Committed: Thu Aug 11 21:58:50 2016 +0200

--
 .../component/salesforce/api/utils/DateTimeUtils.java   | 12 +++-
 .../salesforce/api/utils/DateTimeUtilsTest.groovy   |  2 --
 2 files changed, 7 insertions(+), 7 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/753100b4/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtils.java
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtils.java
 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtils.java
index a5c0da2..fbdcb1a 100644
--- 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtils.java
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtils.java
@@ -21,6 +21,7 @@ import java.time.ZonedDateTime;
 import java.time.format.DateTimeFormatter;
 import java.time.format.DateTimeFormatterBuilder;
 import java.time.format.DateTimeParseException;
+import java.time.format.ResolverStyle;
 import java.util.regex.Pattern;
 
 import static java.time.format.DateTimeFormatter.ISO_LOCAL_DATE_TIME;
@@ -33,11 +34,12 @@ public abstract class DateTimeUtils {
 private static final Pattern BAD_TZ_PATTERN = 
Pattern.compile("[+-][0-9]{4}+$");
 
 private static final DateTimeFormatter ISO_8601_FORMATTER = new 
DateTimeFormatterBuilder()
-.parseCaseInsensitive()
-.append(ISO_LOCAL_DATE_TIME)
-.appendPattern("[.S][,S]")
-.appendOffsetId()
-.toFormatter();
+.parseCaseInsensitive()
+.appendPattern("-MM-dd")
+.appendLiteral('T')
+.appendPattern("HH:mm:ss[.SSS]")
+.appendOffset("+HH:MM", "Z")
+.toFormatter();
 
 public static String formatDateTime(ZonedDateTime dateTime) throws 
DateTimeException {
 return ISO_8601_FORMATTER.format(dateTime);

http://git-wip-us.apache.org/repos/asf/camel/blob/753100b4/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
 
b/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
index 3495d08..8452fe0 100644
--- 
a/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
@@ -25,7 +25,6 @@ import java.time.ZonedDateTime
 
 class DateTimeUtilsTest extends Specification {
 @Unroll
-@Ignore
 def "Date #zonedDateTime should render as #result"() {
 expect:
 DateTimeUtils.formatDateTime(zonedDateTime) == result
@@ -39,7 +38,6 @@ class DateTimeUtilsTest extends Specification {
 }
 
 @Unroll
-@Ignore
 def "Date #dateAsString should parse to #result"() {
 expect:
 DateTimeUtils.parseDateTime(dateAsString) == result



camel git commit: CAMEL-10090: added spock unittest for DateTimeUtils with Ignore

2016-08-02 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 104b4258e -> ef2bd37c2


CAMEL-10090: added spock unittest for DateTimeUtils with Ignore


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/ef2bd37c
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/ef2bd37c
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/ef2bd37c

Branch: refs/heads/master
Commit: ef2bd37c2ccca1bfece2f6dbd4ce66f5c85cb413
Parents: 104b425
Author: Arno Noordover 
Authored: Tue Aug 2 21:06:57 2016 +0200
Committer: Arno Noordover 
Committed: Tue Aug 2 21:06:57 2016 +0200

--
 .../camel-salesforce-component/pom.xml  | 23 
 .../api/utils/DateTimeUtilsTest.groovy  | 58 
 2 files changed, 81 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/ef2bd37c/components/camel-salesforce/camel-salesforce-component/pom.xml
--
diff --git a/components/camel-salesforce/camel-salesforce-component/pom.xml 
b/components/camel-salesforce/camel-salesforce-component/pom.xml
index 3f20a73..29171ec 100644
--- a/components/camel-salesforce/camel-salesforce-component/pom.xml
+++ b/components/camel-salesforce/camel-salesforce-component/pom.xml
@@ -41,6 +41,12 @@
 
   
 
+  org.spockframework
+  spock-core
+  1.0-groovy-2.4
+  test
+
+
   commons-io
   commons-io
   test
@@ -148,4 +154,21 @@
 
   
 
+  
+
+  
+org.codehaus.gmavenplus
+gmavenplus-plugin
+1.4
+
+  
+
+  compile
+  testCompile
+
+  
+
+  
+
+  
 

http://git-wip-us.apache.org/repos/asf/camel/blob/ef2bd37c/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
 
b/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
new file mode 100644
index 000..3495d08
--- /dev/null
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/api/utils/DateTimeUtilsTest.groovy
@@ -0,0 +1,58 @@
+/**
+ * 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.salesforce.api.utils
+
+import spock.lang.Ignore
+import spock.lang.Specification
+import spock.lang.Unroll
+
+import java.time.ZoneId
+import java.time.ZonedDateTime
+
+class DateTimeUtilsTest extends Specification {
+@Unroll
+@Ignore
+def "Date #zonedDateTime should render as #result"() {
+expect:
+DateTimeUtils.formatDateTime(zonedDateTime) == result
+
+where:
+zonedDateTime  
|| result
+ZonedDateTime.of(1991, 12, 10, 12, 13, 14, 700, 
ZoneId.of("UTC+01:00:21")) || "1991-12-10T12:13:14.007+01:00"
+ZonedDateTime.of(1991, 12, 10, 12, 13, 14, 700, ZoneId.of("UTC"))  
|| "1991-12-10T12:13:14.007Z"
+ZonedDateTime.of(1700, 1, 1, 1, 13, 14, 700, 
ZoneId.of("UTC+00:19:21"))|| "1700-01-01T01:13:14.007+00:19"
+ZonedDateTime.of(1700, 2, 3, 2, 13, 14, 700, ZoneId.of("UTC")) 
|| "1700-02-03T02:13:14.007Z"
+}
+
+@Unroll
+@Ignore
+def "Date #dateAsString should parse to #result"() {
+expect:
+DateTimeUtils.parseDateTime(dateAsString) == result
+
+where:
+dateAsString|| result
+"1991-12-10T12:13:14.007+01:00" || ZonedDateTime.of(1991, 12, 10, 12, 
13, 14, 700, ZoneId.of("+01:00"))
+"1991-12-10T12:13:14+00:00" || ZonedDateTime.of(1991, 12, 10, 12, 
13, 14, 0, ZoneId.of("Z"))
+ 

camel git commit: CAMEL-10116: Retrieve last MessageHistory when getting NodeId and RouteId

2016-07-07 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/camel-2.17.x 9e2f79c1d -> 8f3224356


CAMEL-10116: Retrieve last MessageHistory when getting NodeId and RouteId


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/8f322435
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/8f322435
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/8f322435

Branch: refs/heads/camel-2.17.x
Commit: 8f322435627e4c3c5deea41fb760cd6ed85e9fa8
Parents: 9e2f79c
Author: Arno Noordover 
Authored: Tue Jul 5 20:24:35 2016 +0200
Committer: Arno Noordover 
Committed: Thu Jul 7 23:14:19 2016 +0200

--
 .../impl/DefaultAsyncProcessorAwaitManager.java |  46 --
 .../org/apache/camel/impl/DefaultExchange.java  |   3 +-
 .../camel/impl/DefaultInflightRepository.java   |  13 +-
 .../camel/processor/CamelInternalProcessor.java |   3 +-
 .../org/apache/camel/util/ExchangeHelper.java   |   4 +-
 .../DefaultAsyncProcessorAwaitManagerTest.java  | 154 +++
 .../async/AsyncProcessorAwaitManagerTest.java   |   3 +-
 7 files changed, 201 insertions(+), 25 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/8f322435/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
--
diff --git 
a/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
 
b/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
index 2712178..d0d306b 100644
--- 
a/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
+++ 
b/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
@@ -18,7 +18,7 @@ package org.apache.camel.impl;
 
 import java.util.Collection;
 import java.util.Collections;
-import java.util.List;
+import java.util.LinkedList;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.CountDownLatch;
@@ -27,6 +27,7 @@ import java.util.concurrent.atomic.AtomicLong;
 
 import org.apache.camel.Exchange;
 import org.apache.camel.MessageHistory;
+import org.apache.camel.NamedNode;
 import org.apache.camel.processor.DefaultExchangeFormatter;
 import org.apache.camel.spi.AsyncProcessorAwaitManager;
 import org.apache.camel.spi.ExchangeFormatter;
@@ -240,23 +241,12 @@ public class DefaultAsyncProcessorAwaitManager extends 
ServiceSupport implements
 private final Exchange exchange;
 private final CountDownLatch latch;
 private final long start;
-private String routeId;
-private String nodeId;
 
 private AwaitThreadEntry(Thread thread, Exchange exchange, 
CountDownLatch latch) {
 this.thread = thread;
 this.exchange = exchange;
 this.latch = latch;
 this.start = System.currentTimeMillis();
-
-// capture details from message history if enabled
-List list = 
exchange.getProperty(Exchange.MESSAGE_HISTORY, List.class);
-if (list != null && !list.isEmpty()) {
-// grab last part
-MessageHistory history = list.get(list.size() - 1);
-routeId = history.getRouteId();
-nodeId = history.getNode() != null ? history.getNode().getId() 
: null;
-}
 }
 
 @Override
@@ -276,18 +266,46 @@ public class DefaultAsyncProcessorAwaitManager extends 
ServiceSupport implements
 
 @Override
 public String getRouteId() {
-return routeId;
+MessageHistory lastMessageHistory = getLastMessageHistory();
+if (lastMessageHistory == null) {
+return null;
+}
+return lastMessageHistory.getRouteId();
 }
 
 @Override
 public String getNodeId() {
-return nodeId;
+NamedNode node = getNode();
+if (node == null) {
+return null;
+}
+return node.getId();
 }
 
 public CountDownLatch getLatch() {
 return latch;
 }
 
+private NamedNode getNode() {
+MessageHistory lastMessageHistory = getLastMessageHistory();
+if (lastMessageHistory == null) {
+return null;
+}
+return lastMessageHistory.getNode();
+}
+
+private MessageHistory getLastMessageHistory() {
+LinkedList list = getMessageHistories();
+if (list == null || list.isEmpty()) {
+return null;
+}
+return list.getLast();
+}
+
+private LinkedList getMessageHistories() {
+return exchange.getProperty(Exchange.MESSAGE_HISTORY, 
LinkedList.class);
+}

camel git commit: CAMEL-10116: Retrieve last MessageHistory when getting NodeId and RouteId

2016-07-07 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 6483909c2 -> 946c8dffb


CAMEL-10116: Retrieve last MessageHistory when getting NodeId and RouteId


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/946c8dff
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/946c8dff
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/946c8dff

Branch: refs/heads/master
Commit: 946c8dffb69f5bc95392d3e27aeaf5212cd46c87
Parents: 6483909
Author: Arno Noordover 
Authored: Tue Jul 5 20:24:35 2016 +0200
Committer: Arno Noordover 
Committed: Thu Jul 7 22:42:00 2016 +0200

--
 .../impl/DefaultAsyncProcessorAwaitManager.java |  46 --
 .../org/apache/camel/impl/DefaultExchange.java  |   3 +-
 .../camel/impl/DefaultInflightRepository.java   |  13 +-
 .../camel/processor/CamelInternalProcessor.java |   3 +-
 .../org/apache/camel/util/ExchangeHelper.java   |   4 +-
 .../DefaultAsyncProcessorAwaitManagerTest.java  | 154 +++
 .../async/AsyncProcessorAwaitManagerTest.java   |   3 +-
 7 files changed, 201 insertions(+), 25 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/946c8dff/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
--
diff --git 
a/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
 
b/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
index 2712178..d0d306b 100644
--- 
a/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
+++ 
b/camel-core/src/main/java/org/apache/camel/impl/DefaultAsyncProcessorAwaitManager.java
@@ -18,7 +18,7 @@ package org.apache.camel.impl;
 
 import java.util.Collection;
 import java.util.Collections;
-import java.util.List;
+import java.util.LinkedList;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.CountDownLatch;
@@ -27,6 +27,7 @@ import java.util.concurrent.atomic.AtomicLong;
 
 import org.apache.camel.Exchange;
 import org.apache.camel.MessageHistory;
+import org.apache.camel.NamedNode;
 import org.apache.camel.processor.DefaultExchangeFormatter;
 import org.apache.camel.spi.AsyncProcessorAwaitManager;
 import org.apache.camel.spi.ExchangeFormatter;
@@ -240,23 +241,12 @@ public class DefaultAsyncProcessorAwaitManager extends 
ServiceSupport implements
 private final Exchange exchange;
 private final CountDownLatch latch;
 private final long start;
-private String routeId;
-private String nodeId;
 
 private AwaitThreadEntry(Thread thread, Exchange exchange, 
CountDownLatch latch) {
 this.thread = thread;
 this.exchange = exchange;
 this.latch = latch;
 this.start = System.currentTimeMillis();
-
-// capture details from message history if enabled
-List list = 
exchange.getProperty(Exchange.MESSAGE_HISTORY, List.class);
-if (list != null && !list.isEmpty()) {
-// grab last part
-MessageHistory history = list.get(list.size() - 1);
-routeId = history.getRouteId();
-nodeId = history.getNode() != null ? history.getNode().getId() 
: null;
-}
 }
 
 @Override
@@ -276,18 +266,46 @@ public class DefaultAsyncProcessorAwaitManager extends 
ServiceSupport implements
 
 @Override
 public String getRouteId() {
-return routeId;
+MessageHistory lastMessageHistory = getLastMessageHistory();
+if (lastMessageHistory == null) {
+return null;
+}
+return lastMessageHistory.getRouteId();
 }
 
 @Override
 public String getNodeId() {
-return nodeId;
+NamedNode node = getNode();
+if (node == null) {
+return null;
+}
+return node.getId();
 }
 
 public CountDownLatch getLatch() {
 return latch;
 }
 
+private NamedNode getNode() {
+MessageHistory lastMessageHistory = getLastMessageHistory();
+if (lastMessageHistory == null) {
+return null;
+}
+return lastMessageHistory.getNode();
+}
+
+private MessageHistory getLastMessageHistory() {
+LinkedList list = getMessageHistories();
+if (list == null || list.isEmpty()) {
+return null;
+}
+return list.getLast();
+}
+
+private LinkedList getMessageHistories() {
+return exchange.getProperty(Exchange.MESSAGE_HISTORY, 
LinkedList.class);
+}
+
 @

camel git commit: CAMEL-10121: don't log error when an exception is caught in the ResponseHandler. This closes #1064

2016-07-05 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/camel-2.16.x 1565b4ceb -> 64f30474c


CAMEL-10121: don't log error when an exception is caught in the 
ResponseHandler. This closes #1064


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/64f30474
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/64f30474
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/64f30474

Branch: refs/heads/camel-2.16.x
Commit: 64f30474c34cc619631163eaf87ade31da1ce2b0
Parents: 1565b4c
Author: Gijsbert van den Brink 
Authored: Tue Jul 5 20:53:09 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jul 6 00:46:50 2016 +0200

--
 .../main/java/org/apache/camel/component/mina2/Mina2Producer.java  | 2 --
 1 file changed, 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/64f30474/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
--
diff --git 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
index 132abe3..4b43de3 100644
--- 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
+++ 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
@@ -506,8 +506,6 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 
 @Override
 public void exceptionCaught(IoSession ioSession, Throwable cause) {
-LOG.error("Exception on receiving message from address: " + address
-  + " using connector: " + connector, cause);
 this.message = null;
 this.messageReceived = false;
 this.cause = cause;



camel git commit: CAMEL-10121: don't log error when an exception is caught in the ResponseHandler. This closes #1064

2016-07-05 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/camel-2.17.x 19fb54dc5 -> 3e0cd7e79


CAMEL-10121: don't log error when an exception is caught in the 
ResponseHandler. This closes #1064


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/3e0cd7e7
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/3e0cd7e7
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/3e0cd7e7

Branch: refs/heads/camel-2.17.x
Commit: 3e0cd7e796351bdf2418a088140b69a106db8080
Parents: 19fb54d
Author: Gijsbert van den Brink 
Authored: Tue Jul 5 20:53:09 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jul 6 00:43:35 2016 +0200

--
 .../main/java/org/apache/camel/component/mina2/Mina2Producer.java  | 2 --
 1 file changed, 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/3e0cd7e7/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
--
diff --git 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
index 8e3eaec..c5e9c63 100644
--- 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
+++ 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
@@ -508,8 +508,6 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 
 @Override
 public void exceptionCaught(IoSession ioSession, Throwable cause) {
-LOG.error("Exception on receiving message from address: " + address
-  + " using connector: " + connector, cause);
 this.message = null;
 this.messageReceived = false;
 this.cause = cause;



camel git commit: CAMEL-10121: don't log error when an exception is caught in the ResponseHandler. This closes #1064

2016-07-05 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master da59c9745 -> 149e3008e


CAMEL-10121: don't log error when an exception is caught in the 
ResponseHandler. This closes #1064


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/149e3008
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/149e3008
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/149e3008

Branch: refs/heads/master
Commit: 149e3008ec4ce7edcebd300852978a79d751e6a5
Parents: da59c97
Author: Gijsbert van den Brink 
Authored: Tue Jul 5 20:53:09 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jul 6 00:36:41 2016 +0200

--
 .../main/java/org/apache/camel/component/mina2/Mina2Producer.java  | 2 --
 1 file changed, 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/149e3008/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
--
diff --git 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
index 2ac6e92..fc7b4a2 100644
--- 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
+++ 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
@@ -508,8 +508,6 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 
 @Override
 public void exceptionCaught(IoSession ioSession, Throwable cause) {
-LOG.error("Exception on receiving message from address: " + address
-  + " using connector: " + connector, cause);
 this.message = null;
 this.messageReceived = false;
 this.cause = cause;



camel git commit: CAMEL-10115: introduced pollTimeoutMs with default 5000

2016-07-04 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 3b508f55d -> 3485b6152


CAMEL-10115: introduced pollTimeoutMs with default 5000


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/3485b615
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/3485b615
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/3485b615

Branch: refs/heads/master
Commit: 3485b61520d0f3d2823f2fb689ad75600f4d3cfd
Parents: 3b508f5
Author: Arno Noordover 
Authored: Sun Jul 3 20:30:03 2016 +0200
Committer: Arno Noordover 
Committed: Mon Jul 4 14:24:17 2016 +0200

--
 components/camel-kafka/src/main/docs/kafka.adoc |  7 ++-
 .../component/kafka/KafkaConfiguration.java | 15 +-
 .../camel/component/kafka/KafkaConsumer.java|  4 +-
 .../component/kafka/KafkaConsumerTest.java  | 12 +++--
 .../clients/consumer/KafkaConsumerTest.java | 52 
 5 files changed, 83 insertions(+), 7 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/3485b615/components/camel-kafka/src/main/docs/kafka.adoc
--
diff --git a/components/camel-kafka/src/main/docs/kafka.adoc 
b/components/camel-kafka/src/main/docs/kafka.adoc
index 557eec2..575c6a2 100644
--- a/components/camel-kafka/src/main/docs/kafka.adoc
+++ b/components/camel-kafka/src/main/docs/kafka.adoc
@@ -87,8 +87,10 @@ The Kafka component supports 1 options which are listed 
below.
 
 
 
+
+
 // endpoint options: START
-The Kafka component supports 73 endpoint options which are listed below:
+The Kafka component supports 74 endpoint options which are listed below:
 
 {% raw %}
 [width="100%",cols="2s,1,1m,1m,5",options="header"]
@@ -115,6 +117,7 @@ The Kafka component supports 73 endpoint options which are 
listed below:
 | keyDeserializer | consumer | 
org.apache.kafka.common.serialization.StringDeserializer | String | 
Deserializer class for key that implements the Deserializer interface.
 | maxPartitionFetchBytes | consumer | 1048576 | Integer | The maximum amount 
of data per-partition the server will return. The maximum total memory used for 
a request will be partitions max.partition.fetch.bytes. This size must be at 
least as large as the maximum message size the server allows or else it is 
possible for the producer to send messages larger than the consumer can fetch. 
If that happens the consumer can get stuck trying to fetch a large message on a 
certain partition.
 | partitionAssignor | consumer | 
org.apache.kafka.clients.consumer.RangeAssignor | String | The class name of 
the partition assignment strategy that the client will use to distribute 
partition ownership amongst consumer instances when group management is used
+| pollTimeoutMs | consumer | 5000 | Long | The timeout used when polling the 
KafkaConsumer.
 | seekToBeginning | consumer | false | boolean | If the option is true then 
KafkaConsumer will read from beginning on startup.
 | sessionTimeoutMs | consumer | 3 | Integer | The timeout used to detect 
failures when using Kafka's group management facilities.
 | valueDeserializer | consumer | 
org.apache.kafka.common.serialization.StringDeserializer | String | 
Deserializer class for value that implements the Deserializer interface.
@@ -180,6 +183,8 @@ The Kafka component supports 73 endpoint options which are 
listed below:
 
 
 
+
+
 For more information about Producer/Consumer configuration:
 
 
http://kafka.apache.org/documentation.html#newconsumerconfigs[http://kafka.apache.org/documentation.html#newconsumerconfigs]

http://git-wip-us.apache.org/repos/asf/camel/blob/3485b615/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
--
diff --git 
a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
 
b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
index 35ebd36..c69f32f 100644
--- 
a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
+++ 
b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
@@ -71,7 +71,9 @@ public class KafkaConfiguration {
 //session.timeout.ms
 @UriParam(label = "consumer", defaultValue = "3")
 private Integer sessionTimeoutMs = 3;
-//auto.offset.reset
+@UriParam(label = "consumer", defaultValue = "5000")
+private Long pollTimeoutMs = 5000L;
+//auto.offset.reset1
 @UriParam(label = "consumer", defaultValue = "latest", enums = 
"latest,earliest,none")
 private String autoOffsetReset = "latest";
 //partition.assignment.strategy
@@ -1074,6 +1076,17 @@ public class KafkaConfiguration {
 this.sessio

[camel] Git Push Summary

2016-07-04 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/CAMEL-10115 [deleted] e27af6d81


camel git commit: CAMEL-10115: introduced pollTimeoutMs with default 5000

2016-07-04 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/CAMEL-10115 [created] e27af6d81


CAMEL-10115: introduced pollTimeoutMs with default 5000


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/e27af6d8
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/e27af6d8
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/e27af6d8

Branch: refs/heads/CAMEL-10115
Commit: e27af6d815e683b7b2d046a4b7c54bd625e9bcf8
Parents: 16cd1a7
Author: Arno Noordover 
Authored: Sun Jul 3 20:30:03 2016 +0200
Committer: Arno Noordover 
Committed: Mon Jul 4 13:54:49 2016 +0200

--
 components/camel-kafka/src/main/docs/kafka.adoc |  7 ++-
 .../component/kafka/KafkaConfiguration.java | 15 +-
 .../camel/component/kafka/KafkaConsumer.java|  4 +-
 .../component/kafka/KafkaConsumerTest.java  | 12 +++--
 .../clients/consumer/KafkaConsumerTest.java | 52 
 5 files changed, 83 insertions(+), 7 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/e27af6d8/components/camel-kafka/src/main/docs/kafka.adoc
--
diff --git a/components/camel-kafka/src/main/docs/kafka.adoc 
b/components/camel-kafka/src/main/docs/kafka.adoc
index 557eec2..575c6a2 100644
--- a/components/camel-kafka/src/main/docs/kafka.adoc
+++ b/components/camel-kafka/src/main/docs/kafka.adoc
@@ -87,8 +87,10 @@ The Kafka component supports 1 options which are listed 
below.
 
 
 
+
+
 // endpoint options: START
-The Kafka component supports 73 endpoint options which are listed below:
+The Kafka component supports 74 endpoint options which are listed below:
 
 {% raw %}
 [width="100%",cols="2s,1,1m,1m,5",options="header"]
@@ -115,6 +117,7 @@ The Kafka component supports 73 endpoint options which are 
listed below:
 | keyDeserializer | consumer | 
org.apache.kafka.common.serialization.StringDeserializer | String | 
Deserializer class for key that implements the Deserializer interface.
 | maxPartitionFetchBytes | consumer | 1048576 | Integer | The maximum amount 
of data per-partition the server will return. The maximum total memory used for 
a request will be partitions max.partition.fetch.bytes. This size must be at 
least as large as the maximum message size the server allows or else it is 
possible for the producer to send messages larger than the consumer can fetch. 
If that happens the consumer can get stuck trying to fetch a large message on a 
certain partition.
 | partitionAssignor | consumer | 
org.apache.kafka.clients.consumer.RangeAssignor | String | The class name of 
the partition assignment strategy that the client will use to distribute 
partition ownership amongst consumer instances when group management is used
+| pollTimeoutMs | consumer | 5000 | Long | The timeout used when polling the 
KafkaConsumer.
 | seekToBeginning | consumer | false | boolean | If the option is true then 
KafkaConsumer will read from beginning on startup.
 | sessionTimeoutMs | consumer | 3 | Integer | The timeout used to detect 
failures when using Kafka's group management facilities.
 | valueDeserializer | consumer | 
org.apache.kafka.common.serialization.StringDeserializer | String | 
Deserializer class for value that implements the Deserializer interface.
@@ -180,6 +183,8 @@ The Kafka component supports 73 endpoint options which are 
listed below:
 
 
 
+
+
 For more information about Producer/Consumer configuration:
 
 
http://kafka.apache.org/documentation.html#newconsumerconfigs[http://kafka.apache.org/documentation.html#newconsumerconfigs]

http://git-wip-us.apache.org/repos/asf/camel/blob/e27af6d8/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
--
diff --git 
a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
 
b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
index 35ebd36..c69f32f 100644
--- 
a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
+++ 
b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
@@ -71,7 +71,9 @@ public class KafkaConfiguration {
 //session.timeout.ms
 @UriParam(label = "consumer", defaultValue = "3")
 private Integer sessionTimeoutMs = 3;
-//auto.offset.reset
+@UriParam(label = "consumer", defaultValue = "5000")
+private Long pollTimeoutMs = 5000L;
+//auto.offset.reset1
 @UriParam(label = "consumer", defaultValue = "latest", enums = 
"latest,earliest,none")
 private String autoOffsetReset = "latest";
 //partition.assignment.strategy
@@ -1074,6 +1076,17 @@ public class KafkaConfiguration {
 this

[1/2] camel git commit: Drop jodatime dependency

2016-07-03 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master b21138530 -> ddc89511d


Drop jodatime dependency


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/683b53af
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/683b53af
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/683b53af

Branch: refs/heads/master
Commit: 683b53afeb8da28eab9c21b8156983875c99a6a2
Parents: b211385
Author: Arno Noordover 
Authored: Wed Jun 29 22:32:21 2016 +0200
Committer: Arno Noordover 
Committed: Sun Jul 3 19:14:56 2016 +0200

--
 .../camel-salesforce-component/pom.xml  |   8 -
 .../src/main/docs/salesforce.adoc   | 267 ++-
 .../salesforce/api/JodaTimeConverter.java   |  35 +--
 .../salesforce/api/dto/AbstractSObjectBase.java |  33 +--
 .../dto/analytics/reports/AsyncAttributes.java  |  15 +-
 .../api/dto/analytics/reports/Report.java   |   8 +-
 .../dto/analytics/reports/ReportInstance.java   |  15 +-
 .../internal/joda/DateTimeDeserializer.java |  29 +-
 .../internal/joda/DateTimeModule.java   |   8 +-
 .../internal/joda/DateTimeSerializer.java   |  28 +-
 .../salesforce/RestApiIntegrationTest.java  |   9 +-
 .../salesforce/StreamingApiIntegrationTest.java |   5 +-
 .../salesforce/dto/generated/Tasks__c.java  |  43 +--
 .../processor/JsonRestProcessorTest.java|  11 +-
 .../camel-salesforce-maven-plugin/README.md |   2 +-
 .../apache/camel/maven/CamelSalesforceMojo.java |   9 +-
 16 files changed, 314 insertions(+), 211 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/683b53af/components/camel-salesforce/camel-salesforce-component/pom.xml
--
diff --git a/components/camel-salesforce/camel-salesforce-component/pom.xml 
b/components/camel-salesforce/camel-salesforce-component/pom.xml
index af5507b..3f20a73 100644
--- a/components/camel-salesforce/camel-salesforce-component/pom.xml
+++ b/components/camel-salesforce/camel-salesforce-component/pom.xml
@@ -31,9 +31,6 @@
   Camel Salesforce support
 
   
-
-org.joda.time.*;version="[1.6,3)"
-
 
   org.apache.camel.component.salesforce,
   org.apache.camel.component.salesforce.api.*
@@ -103,11 +100,6 @@
 
   
 
-
-  joda-time
-  joda-time
-  ${jodatime2-bundle-version}
-
 
 
 

http://git-wip-us.apache.org/repos/asf/camel/blob/683b53af/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
 
b/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
index 3f5c9cd..1fb5aa7 100644
--- 
a/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
@@ -199,88 +199,229 @@ Options
 ^^^
 
 
+
 // component options: START
 The Salesforce component supports 16 options which are listed below.
 
 
 
 {% raw %}
-[width="100%",cols="2s,1m,8",options="header"]
-|===
-| Name | Java Type | Description
-| loginConfig | SalesforceLoginConfig | To use the shared 
SalesforceLoginConfig as login configuration
-| config | SalesforceEndpointConfig | To use the shared SalesforceLoginConfig 
as configuration
-| httpClientProperties | Map | Used for configuring HTTP client properties as 
key/value pairs
-| sslContextParameters | SSLContextParameters | To configure security using 
SSLContextParameters
-| httpProxyHost | String | To configure HTTP proxy host
-| httpProxyPort | Integer | To configure HTTP proxy port
-| httpProxyUsername | String | To configure HTTP proxy username
-| httpProxyPassword | String | To configure HTTP proxy password
-| isHttpProxySocks4 | boolean | Enable for Socks4 proxy false by default
-| isHttpProxySecure | boolean | Enable for TLS connections true by default
-| httpProxyIncludedAddresses | Set | HTTP proxy included addresses
-| httpProxyExcludedAddresses | Set | HTTP proxy excluded addresses
-| httpProxyAuthUri | String | HTTP proxy authentication URI
-| httpProxyRealm | String | HTTP proxy authentication realm
-| httpProxyUseDigestAuth | boolean | Use HTTP proxy Digest authentication 
false by default
-| packages | String[] | Package names to scan for DTO classes (multiple 
packages can be separated by comma).
-|===
+
+loginConfig (SalesforceLoginConfig)::
+  To use the shared SalesforceLoginConfig as login configuration
+config (SalesforceEndpointConfig)::

[2/2] camel git commit: salesforce: ignore integrationtest, changed documentation

2016-07-03 Thread anoordover
salesforce: ignore integrationtest, changed documentation


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/ddc89511
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/ddc89511
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/ddc89511

Branch: refs/heads/master
Commit: ddc89511d7bd24930e29b7a2bc695e1bf29c49b6
Parents: 683b53a
Author: Arno Noordover 
Authored: Sun Jul 3 19:42:26 2016 +0200
Committer: Arno Noordover 
Committed: Sun Jul 3 19:42:26 2016 +0200

--
 .../src/main/docs/salesforce.adoc   | 270 +--
 .../salesforce/RestApiIntegrationTest.java  |   2 +-
 2 files changed, 70 insertions(+), 202 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/ddc89511/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
 
b/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
index 1fb5aa7..f931960 100644
--- 
a/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/main/docs/salesforce.adoc
@@ -200,47 +200,33 @@ Options
 
 
 
+
 // component options: START
 The Salesforce component supports 16 options which are listed below.
 
 
 
 {% raw %}
-
-loginConfig (SalesforceLoginConfig)::
-  To use the shared SalesforceLoginConfig as login configuration
-config (SalesforceEndpointConfig)::
-  To use the shared SalesforceLoginConfig as configuration
-httpClientProperties (Map)::
-  Used for configuring HTTP client properties as key/value pairs
-sslContextParameters (SSLContextParameters)::
-  To configure security using SSLContextParameters
-httpProxyHost (String)::
-  To configure HTTP proxy host
-httpProxyPort (Integer)::
-  To configure HTTP proxy port
-httpProxyUsername (String)::
-  To configure HTTP proxy username
-httpProxyPassword (String)::
-  To configure HTTP proxy password
-isHttpProxySocks4 (boolean)::
-  Enable for Socks4 proxy false by default
-isHttpProxySecure (boolean)::
-  Enable for TLS connections true by default
-httpProxyIncludedAddresses (Set)::
-  HTTP proxy included addresses
-httpProxyExcludedAddresses (Set)::
-  HTTP proxy excluded addresses
-httpProxyAuthUri (String)::
-  HTTP proxy authentication URI
-httpProxyRealm (String)::
-  HTTP proxy authentication realm
-httpProxyUseDigestAuth (boolean)::
-  Use HTTP proxy Digest authentication false by default
-packages (String[])::
-  Package names to scan for DTO classes (multiple packages can be separated by 
comma).
-
-
+[width="100%",cols="2s,1m,8",options="header"]
+|===
+| Name | Java Type | Description
+| loginConfig | SalesforceLoginConfig | To use the shared 
SalesforceLoginConfig as login configuration
+| config | SalesforceEndpointConfig | To use the shared SalesforceLoginConfig 
as configuration
+| httpClientProperties | Map | Used for configuring HTTP client properties as 
key/value pairs
+| sslContextParameters | SSLContextParameters | To configure security using 
SSLContextParameters
+| httpProxyHost | String | To configure HTTP proxy host
+| httpProxyPort | Integer | To configure HTTP proxy port
+| httpProxyUsername | String | To configure HTTP proxy username
+| httpProxyPassword | String | To configure HTTP proxy password
+| isHttpProxySocks4 | boolean | Enable for Socks4 proxy false by default
+| isHttpProxySecure | boolean | Enable for TLS connections true by default
+| httpProxyIncludedAddresses | Set | HTTP proxy included addresses
+| httpProxyExcludedAddresses | Set | HTTP proxy excluded addresses
+| httpProxyAuthUri | String | HTTP proxy authentication URI
+| httpProxyRealm | String | HTTP proxy authentication realm
+| httpProxyUseDigestAuth | boolean | Use HTTP proxy Digest authentication 
false by default
+| packages | String[] | Package names to scan for DTO classes (multiple 
packages can be separated by comma).
+|===
 {% endraw %}
 // component options: END
 
@@ -248,180 +234,61 @@ packages (String[])::
 
 
 
+
+
 // endpoint options: START
 The Salesforce component supports 39 endpoint options which are listed below:
 
 {% raw %}
-
-common::
-
-operationName (OperationName):::
-
-  
-
-topicName (String):::
-
-  
-
-apexMethod (String):::
-
-  APEX method name
-
-apexQueryParams (Map):::
-
-  Query params for APEX method
-
-apexUrl (String):::
-
-  APEX method URL
-
-apiVersion (String):::
-
-  Salesforce API version defaults to SalesforceEndpointConfig.DEFAULT_VERSION
-
-batchId (String):::
-
-  Bulk AP

[2/2] camel git commit: CAMEL-9467: remove deprecated config-items

2016-07-02 Thread anoordover
CAMEL-9467: remove deprecated config-items


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/4003fc7f
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/4003fc7f
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/4003fc7f

Branch: refs/heads/master
Commit: 4003fc7f4f4d09fb8b91a5ee2e78c562487b6aad
Parents: 5c2f89b
Author: Arno Noordover 
Authored: Sat Jul 2 17:26:16 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jul 2 17:26:16 2016 +0200

--
 components/camel-kafka/src/main/docs/kafka.adoc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/4003fc7f/components/camel-kafka/src/main/docs/kafka.adoc
--
diff --git a/components/camel-kafka/src/main/docs/kafka.adoc 
b/components/camel-kafka/src/main/docs/kafka.adoc
index 8a07a13..557eec2 100644
--- a/components/camel-kafka/src/main/docs/kafka.adoc
+++ b/components/camel-kafka/src/main/docs/kafka.adoc
@@ -86,8 +86,9 @@ The Kafka component supports 1 options which are listed below.
 
 
 
+
 // endpoint options: START
-The Kafka component supports 75 endpoint options which are listed below:
+The Kafka component supports 73 endpoint options which are listed below:
 
 {% raw %}
 [width="100%",cols="2s,1,1m,1m,5",options="header"]
@@ -102,8 +103,6 @@ The Kafka component supports 75 endpoint options which are 
listed below:
 | autoCommitEnable | consumer | true | Boolean | If true periodically commit 
to ZooKeeper the offset of messages already fetched by the consumer. This 
committed offset will be used when the process fails as the position from which 
the new consumer will begin.
 | autoCommitIntervalMs | consumer | 5000 | Integer | The frequency in ms that 
the consumer offsets are committed to zookeeper.
 | autoOffsetReset | consumer | latest | String | What to do when there is no 
initial offset in ZooKeeper or if an offset is out of range: smallest : 
automatically reset the offset to the smallest offset largest : automatically 
reset the offset to the largest offset fail: throw exception to the consumer
-| barrierAwaitTimeoutMs | consumer | 1 | int | If the BatchingConsumerTask 
processes exchange exceed the batchSize it will wait for barrierAwaitTimeoutMs.
-| batchSize | consumer | 100 | int | The batchSize that the 
BatchingConsumerTask processes once.
 | bridgeErrorHandler | consumer | false | boolean | Allows for bridging the 
consumer to the Camel routing Error Handler which mean any exceptions occurred 
while the consumer is trying to pickup incoming messages or the likes will now 
be processed as a message and handled by the routing Error Handler. By default 
the consumer will use the org.apache.camel.spi.ExceptionHandler to deal with 
exceptions that will be logged at WARN/ERROR level and ignored.
 | checkCrcs | consumer | true | Boolean | Automatically check the CRC32 of the 
records consumed. This ensures no on-the-wire or on-disk corruption to the 
messages occurred. This check adds some overhead so it may be disabled in cases 
seeking extreme performance.
 | consumerId | consumer |  | String | Generated automatically if not set.
@@ -180,6 +179,7 @@ The Kafka component supports 75 endpoint options which are 
listed below:
 
 
 
+
 For more information about Producer/Consumer configuration:
 
 
http://kafka.apache.org/documentation.html#newconsumerconfigs[http://kafka.apache.org/documentation.html#newconsumerconfigs]



[1/2] camel git commit: CAMEL-9467: remove deprecated config-items

2016-07-02 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 8b6ff3f77 -> 4003fc7f4


CAMEL-9467: remove deprecated config-items


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/5c2f89b9
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/5c2f89b9
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/5c2f89b9

Branch: refs/heads/master
Commit: 5c2f89b90d94e21ac336bc2dd22b03292634a7a3
Parents: 8b6ff3f
Author: Arno Noordover 
Authored: Sat Jul 2 17:25:47 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jul 2 17:25:47 2016 +0200

--
 .../component/kafka/KafkaConfiguration.java | 26 
 .../camel/component/kafka/KafkaEndpoint.java| 16 
 .../kafka/KafkaConsumerBatchSizeTest.java   |  2 --
 3 files changed, 44 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/5c2f89b9/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
--
diff --git 
a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
 
b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
index 7a7f9d4..35ebd36 100644
--- 
a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
+++ 
b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java
@@ -48,10 +48,6 @@ public class KafkaConfiguration {
 private int consumerStreams = 10;
 @UriParam(label = "consumer", defaultValue = "1")
 private int consumersCount = 1;
-@UriParam(label = "consumer", defaultValue = "100")
-private int batchSize = 100;
-@UriParam(label = "consumer", defaultValue = "1")
-private int barrierAwaitTimeoutMs = 1;
 
 //Common configuration properties
 @UriParam
@@ -407,28 +403,6 @@ public class KafkaConfiguration {
 this.consumerStreams = consumerStreams;
 }
 
-public int getBatchSize() {
-return batchSize;
-}
-
-/**
- * The batchSize that the BatchingConsumerTask processes once.
- */
-public void setBatchSize(int batchSize) {
-this.batchSize = batchSize;
-}
-
-public int getBarrierAwaitTimeoutMs() {
-return barrierAwaitTimeoutMs;
-}
-
-/**
- * If the BatchingConsumerTask processes exchange exceed the batchSize, it 
will wait for barrierAwaitTimeoutMs.
- */
-public void setBarrierAwaitTimeoutMs(int barrierAwaitTimeoutMs) {
-this.barrierAwaitTimeoutMs = barrierAwaitTimeoutMs;
-}
-
 public int getConsumersCount() {
 return consumersCount;
 }

http://git-wip-us.apache.org/repos/asf/camel/blob/5c2f89b9/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaEndpoint.java
--
diff --git 
a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaEndpoint.java
 
b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaEndpoint.java
index 74aa883..e918bfd 100644
--- 
a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaEndpoint.java
+++ 
b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaEndpoint.java
@@ -374,10 +374,6 @@ public class KafkaEndpoint extends DefaultEndpoint 
implements MultipleConsumersS
 return configuration.getSslTruststoreLocation();
 }
 
-public void setBarrierAwaitTimeoutMs(int barrierAwaitTimeoutMs) {
-configuration.setBarrierAwaitTimeoutMs(barrierAwaitTimeoutMs);
-}
-
 public String getSslProvider() {
 return configuration.getSslProvider();
 }
@@ -398,10 +394,6 @@ public class KafkaEndpoint extends DefaultEndpoint 
implements MultipleConsumersS
 return configuration.getTopic();
 }
 
-public int getBarrierAwaitTimeoutMs() {
-return configuration.getBarrierAwaitTimeoutMs();
-}
-
 public Integer getFetchMinBytes() {
 return configuration.getFetchMinBytes();
 }
@@ -462,10 +454,6 @@ public class KafkaEndpoint extends DefaultEndpoint 
implements MultipleConsumersS
 configuration.setConsumersCount(consumersCount);
 }
 
-public int getBatchSize() {
-return configuration.getBatchSize();
-}
-
 public void setAutoCommitIntervalMs(Integer autoCommitIntervalMs) {
 configuration.setAutoCommitIntervalMs(autoCommitIntervalMs);
 }
@@ -626,10 +614,6 @@ public class KafkaEndpoint extends DefaultEndpoint 
implements MultipleConsumersS
 return configuration.getSaslKerberosServiceName();
 }
 
-public void setBatchSize(int batchSize) {
-configuration.setBatchSize(batchSize);
-}
-
   

camel git commit: ITest: dynamic port for activemq

2016-07-02 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 85b5e05f6 -> 8b6ff3f77


ITest: dynamic port for activemq


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/8b6ff3f7
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/8b6ff3f7
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/8b6ff3f7

Branch: refs/heads/master
Commit: 8b6ff3f773930055f87b25e8d08754a818a917a2
Parents: 85b5e05
Author: Arno Noordover 
Authored: Sat Jul 2 15:39:46 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jul 2 15:39:46 2016 +0200

--
 tests/camel-itest/pom.xml   |  3 +-
 .../org/apache/camel/itest/ITestSupport.java| 63 
 ...romJmsToJdbcIdempotentConsumerToJmsTest.java | 46 +++---
 ...mJmsToJdbcIdempotentConsumerToJmsXaTest.java |  2 +
 .../camel/itest/tx/Jms2RequiresNewTest.java |  2 +
 .../camel-itest/src/test/resources/activemq.xml |  7 ++-
 ...FromJmsToJdbcIdempotentConsumerToJmsTest.xml | 22 +++
 ...omJmsToJdbcIdempotentConsumerToJmsXaTest.xml |  8 ++-
 .../itest/tx/Jms2RequiresNewTest-context.xml|  6 +-
 9 files changed, 119 insertions(+), 40 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/8b6ff3f7/tests/camel-itest/pom.xml
--
diff --git a/tests/camel-itest/pom.xml b/tests/camel-itest/pom.xml
index d96394e..be5c947 100644
--- a/tests/camel-itest/pom.xml
+++ b/tests/camel-itest/pom.xml
@@ -409,7 +409,8 @@
 maven-surefire-plugin
 
   
-  always
+  1
+  false
   3000
   
 target/derby.log

http://git-wip-us.apache.org/repos/asf/camel/blob/8b6ff3f7/tests/camel-itest/src/test/java/org/apache/camel/itest/ITestSupport.java
--
diff --git 
a/tests/camel-itest/src/test/java/org/apache/camel/itest/ITestSupport.java 
b/tests/camel-itest/src/test/java/org/apache/camel/itest/ITestSupport.java
new file mode 100644
index 000..3cde0b5
--- /dev/null
+++ b/tests/camel-itest/src/test/java/org/apache/camel/itest/ITestSupport.java
@@ -0,0 +1,63 @@
+/**
+ * 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.itest;
+
+import org.apache.camel.test.AvailablePortFinder;
+
+/**
+ * For test cases that use unique contexts, they can share the 
+ * ports which will make things a bit faster as ports aren't opened
+ * and closed all the time. 
+ */
+public final class ITestSupport {
+
+static final int PORT1 = AvailablePortFinder.getNextAvailable();
+static final int PORT2 = AvailablePortFinder.getNextAvailable();
+static final int PORT3 = AvailablePortFinder.getNextAvailable(61616);
+static final int PORT4 = AvailablePortFinder.getNextAvailable(61616);
+
+static {
+//set them as system properties so Spring can use the property 
placeholder
+//things to set them into the URL's in the spring contexts
+System.setProperty("ITestSupport.port1", Integer.toString(PORT1));
+System.setProperty("ITestSupport.port2", Integer.toString(PORT2));
+System.setProperty("ITestSupport.port3", Integer.toString(PORT3));
+System.setProperty("ITestSupport.port4", Integer.toString(PORT4));
+}
+
+private ITestSupport() {
+}
+
+public static int getPort(String name) {
+int port = AvailablePortFinder.getNextAvailable();
+System.setProperty(name, Integer.toString(port));
+return port;
+}
+
+public static int getPort1() {
+return PORT1;
+}
+
+public static int getPort2() {
+return PORT2;
+}
+
+public static int getPort3() {
+return PORT3;
+}
+}

http://git-wip-us.apache.org/repos/asf/camel/blob/8b6ff3f7/tests/camel-itest/src/test/java/org/apache/camel/itest/sql/FromJmsToJdbcIdempotentConsumerToJmsTest.java
--
diff --git 
a/tests/camel-itest/src/test/java/org/apache/cam

camel git commit: CAMEL-10110: Split unittest so only less asserts are done per test

2016-07-01 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 67b2e6927 -> 0b812ceab


CAMEL-10110: Split unittest so only less asserts are done per test


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/0b812cea
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/0b812cea
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/0b812cea

Branch: refs/heads/master
Commit: 0b812ceabf6fdb7a2a2e4ef5b084d1d79205b80a
Parents: 67b2e69
Author: Arno Noordover 
Authored: Sat Jul 2 01:36:57 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jul 2 01:36:57 2016 +0200

--
 .../bindy/csv/BindyPipeDelimiterTest.java   | 24 ++--
 1 file changed, 17 insertions(+), 7 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/0b812cea/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
--
diff --git 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
index 86b67fb..85c3071 100644
--- 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
+++ 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
@@ -18,13 +18,10 @@ package org.apache.camel.dataformat.bindy.csv;
 
 import java.util.List;
 
-import org.apache.camel.Exchange;
 import org.apache.camel.builder.RouteBuilder;
 import org.apache.camel.component.mock.MockEndpoint;
 import org.apache.camel.dataformat.bindy.model.simple.pipeline.MyData;
 import org.apache.camel.model.dataformat.BindyType;
-import org.apache.camel.processor.aggregate.AggregationStrategy;
-import org.apache.camel.processor.aggregate.GroupedExchangeAggregationStrategy;
 import org.apache.camel.test.junit4.CamelTestSupport;
 import org.junit.Test;
 
@@ -58,11 +55,25 @@ public class BindyPipeDelimiterTest extends 
CamelTestSupport {
 }
 
 @Test
-public void testBindyPipeDelimiterMarshal() throws Exception {
+public void testBindyPipeDelimiterMarshalShouldHaveCorrectHeader() throws 
Exception {
 MockEndpoint mock = getMockEndpoint("mock:result");
 mock.expectedMessageCount(1);
-mock.message(0).body().startsWith("col1|col2|col3");
-//mock.message(0).body().("HAPPY|NEW|YEAR");
+mock.message(0).body().convertToString().startsWith("col1|col2|col3");
+
+MyData data = new MyData();
+data.setCol1("HAPPY");
+data.setCol2("NEW");
+data.setCol3("YEAR");
+template.sendBody("direct:marshal", data);
+
+assertMockEndpointsSatisfied();
+}
+
+@Test
+public void testBindyPipeDelimiterMarshalShouldContainMyData() throws 
Exception {
+MockEndpoint mock = getMockEndpoint("mock:result");
+mock.expectedMessageCount(1);
+mock.message(0).body().convertToString().contains("HAPPY|NEW|YEAR");
 
 MyData data = new MyData();
 data.setCol1("HAPPY");
@@ -71,7 +82,6 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 template.sendBody("direct:marshal", data);
 
 assertMockEndpointsSatisfied();
-System.out.println(mock.message(0).body().convertToString());
 }
 
 @Override



camel git commit: CAMEL-10110: Solved removal of regexp escape char from HEADER

2016-07-01 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/camel-2.16.x ee232060f -> 1565b4ceb


CAMEL-10110: Solved removal of regexp escape char from HEADER


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/1565b4ce
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/1565b4ce
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/1565b4ce

Branch: refs/heads/camel-2.16.x
Commit: 1565b4cebab53b4a0ba9c7129b2cd34b6e1572b9
Parents: ee23206
Author: Arno Noordover 
Authored: Sat Jul 2 01:07:26 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jul 2 01:28:36 2016 +0200

--
 .../camel/dataformat/bindy/BindyCsvFactory.java   |  2 +-
 .../bindy/csv/BindyPipeDelimiterTest.java | 18 +++---
 .../bindy/model/simple/pipeline/MyData.java   |  2 +-
 3 files changed, 13 insertions(+), 9 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/1565b4ce/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
--
diff --git 
a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
 
b/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
index a699c04..0c5a49f 100755
--- 
a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
+++ 
b/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
@@ -504,7 +504,7 @@ public class BindyCsvFactory extends BindyAbstractFactory 
implements BindyFactor
 }
 
 if (it.hasNext()) {
-builderHeader.append(separator);
+
builderHeader.append(ConverterUtils.getCharDelimiter(separator));
 }
 
 }

http://git-wip-us.apache.org/repos/asf/camel/blob/1565b4ce/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
--
diff --git 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
index f6dabc8..790c12e 100644
--- 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
+++ 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
@@ -26,7 +26,7 @@ import org.apache.camel.test.junit4.CamelTestSupport;
 import org.junit.Test;
 
 /**
- * @version 
+ * @version
  */
 public class BindyPipeDelimiterTest extends CamelTestSupport {
 
@@ -41,7 +41,7 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 
 MyData rec1 = (MyData) 
mock.getReceivedExchanges().get(0).getIn().getBody(List.class).get(0);
 MyData rec2 = (MyData) 
mock.getReceivedExchanges().get(0).getIn().getBody(List.class).get(1);
-   
+
 //MyData rec1 = (MyData) map1.values().iterator().next();
 //MyData rec2 = (MyData) map2.values().iterator().next();
 
@@ -58,7 +58,8 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 public void testBindyPipeDelimiterMarshal() throws Exception {
 MockEndpoint mock = getMockEndpoint("mock:result");
 mock.expectedMessageCount(1);
-mock.message(0).body().startsWith("HAPPY|NEW|YEAR");
+mock.message(0).body().startsWith("col1|col2|col3");
+//mock.message(0).body().("HAPPY|NEW|YEAR");
 
 MyData data = new MyData();
 data.setCol1("HAPPY");
@@ -67,6 +68,7 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 template.sendBody("direct:marshal", data);
 
 assertMockEndpointsSatisfied();
+System.out.println(mock.message(0).body().convertToString());
 }
 
 @Override
@@ -75,12 +77,14 @@ public class BindyPipeDelimiterTest extends 
CamelTestSupport {
 @Override
 public void configure() throws Exception {
 from("direct:unmarshal")
-.unmarshal().bindy(BindyType.Csv, 
org.apache.camel.dataformat.bindy.model.simple.pipeline.MyData.class)
-.to("mock:result");
+.unmarshal().bindy(BindyType.Csv, 
org.apache.camel.dataformat.bindy.model.simple.pipeline.MyData.class)
+.to("log:after.unmarshal")
+.to("mock:result");
 
 from("direct:marshal")
-.marshal().bindy(BindyType.Csv, 
org.apache.camel.dataformat.bindy.model.simple.pipeline.MyData.class)
-.to("mock:result");
+.marshal().bindy(BindyType.Csv, 
org.apache.camel

camel git commit: CAMEL-10110: Solved removal of regexp escape char from HEADER

2016-07-01 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/camel-2.17.x b82e6 -> 7b65252a5


CAMEL-10110: Solved removal of regexp escape char from HEADER


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/7b65252a
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/7b65252a
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/7b65252a

Branch: refs/heads/camel-2.17.x
Commit: 7b65252a5bfa909a50aec398585ee6206f214166
Parents: b82e6ee
Author: Arno Noordover 
Authored: Sat Jul 2 01:07:26 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jul 2 01:22:20 2016 +0200

--
 .../camel/dataformat/bindy/BindyCsvFactory.java |  2 +-
 .../bindy/csv/BindyPipeDelimiterTest.java   | 21 +---
 .../bindy/model/simple/pipeline/MyData.java |  2 +-
 3 files changed, 16 insertions(+), 9 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/7b65252a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
--
diff --git 
a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
 
b/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
index a699c04..0c5a49f 100755
--- 
a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
+++ 
b/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
@@ -504,7 +504,7 @@ public class BindyCsvFactory extends BindyAbstractFactory 
implements BindyFactor
 }
 
 if (it.hasNext()) {
-builderHeader.append(separator);
+
builderHeader.append(ConverterUtils.getCharDelimiter(separator));
 }
 
 }

http://git-wip-us.apache.org/repos/asf/camel/blob/7b65252a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
--
diff --git 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
index f6dabc8..86b67fb 100644
--- 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
+++ 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
@@ -18,15 +18,18 @@ package org.apache.camel.dataformat.bindy.csv;
 
 import java.util.List;
 
+import org.apache.camel.Exchange;
 import org.apache.camel.builder.RouteBuilder;
 import org.apache.camel.component.mock.MockEndpoint;
 import org.apache.camel.dataformat.bindy.model.simple.pipeline.MyData;
 import org.apache.camel.model.dataformat.BindyType;
+import org.apache.camel.processor.aggregate.AggregationStrategy;
+import org.apache.camel.processor.aggregate.GroupedExchangeAggregationStrategy;
 import org.apache.camel.test.junit4.CamelTestSupport;
 import org.junit.Test;
 
 /**
- * @version 
+ * @version
  */
 public class BindyPipeDelimiterTest extends CamelTestSupport {
 
@@ -41,7 +44,7 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 
 MyData rec1 = (MyData) 
mock.getReceivedExchanges().get(0).getIn().getBody(List.class).get(0);
 MyData rec2 = (MyData) 
mock.getReceivedExchanges().get(0).getIn().getBody(List.class).get(1);
-   
+
 //MyData rec1 = (MyData) map1.values().iterator().next();
 //MyData rec2 = (MyData) map2.values().iterator().next();
 
@@ -58,7 +61,8 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 public void testBindyPipeDelimiterMarshal() throws Exception {
 MockEndpoint mock = getMockEndpoint("mock:result");
 mock.expectedMessageCount(1);
-mock.message(0).body().startsWith("HAPPY|NEW|YEAR");
+mock.message(0).body().startsWith("col1|col2|col3");
+//mock.message(0).body().("HAPPY|NEW|YEAR");
 
 MyData data = new MyData();
 data.setCol1("HAPPY");
@@ -67,6 +71,7 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 template.sendBody("direct:marshal", data);
 
 assertMockEndpointsSatisfied();
+System.out.println(mock.message(0).body().convertToString());
 }
 
 @Override
@@ -75,12 +80,14 @@ public class BindyPipeDelimiterTest extends 
CamelTestSupport {
 @Override
 public void configure() throws Exception {
 from("direct:unmarshal")
-.unmarshal().bindy(BindyType.Csv, 
org.apache.camel.dataformat.bindy.model.simple.pipeline.MyData.class)
-.to("mock:result");
+.

camel git commit: CAMEL-10110: Solved removal of regexp escape char from HEADER

2016-07-01 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 158b8bb68 -> 67b2e6927


CAMEL-10110: Solved removal of regexp escape char from HEADER


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/67b2e692
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/67b2e692
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/67b2e692

Branch: refs/heads/master
Commit: 67b2e69274335953d4c56be00818cbaf134f276b
Parents: 158b8bb
Author: Arno Noordover 
Authored: Sat Jul 2 01:07:26 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jul 2 01:07:26 2016 +0200

--
 .../camel/dataformat/bindy/BindyCsvFactory.java |  2 +-
 .../bindy/csv/BindyPipeDelimiterTest.java   | 21 +---
 .../bindy/model/simple/pipeline/MyData.java |  2 +-
 3 files changed, 16 insertions(+), 9 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/67b2e692/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
--
diff --git 
a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
 
b/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
index e92bdc5..943f85c 100755
--- 
a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
+++ 
b/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyCsvFactory.java
@@ -515,7 +515,7 @@ public class BindyCsvFactory extends BindyAbstractFactory 
implements BindyFactor
 }
 
 if (it.hasNext()) {
-builderHeader.append(separator);
+
builderHeader.append(ConverterUtils.getCharDelimiter(separator));
 }
 
 }

http://git-wip-us.apache.org/repos/asf/camel/blob/67b2e692/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
--
diff --git 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
index f6dabc8..86b67fb 100644
--- 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
+++ 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/csv/BindyPipeDelimiterTest.java
@@ -18,15 +18,18 @@ package org.apache.camel.dataformat.bindy.csv;
 
 import java.util.List;
 
+import org.apache.camel.Exchange;
 import org.apache.camel.builder.RouteBuilder;
 import org.apache.camel.component.mock.MockEndpoint;
 import org.apache.camel.dataformat.bindy.model.simple.pipeline.MyData;
 import org.apache.camel.model.dataformat.BindyType;
+import org.apache.camel.processor.aggregate.AggregationStrategy;
+import org.apache.camel.processor.aggregate.GroupedExchangeAggregationStrategy;
 import org.apache.camel.test.junit4.CamelTestSupport;
 import org.junit.Test;
 
 /**
- * @version 
+ * @version
  */
 public class BindyPipeDelimiterTest extends CamelTestSupport {
 
@@ -41,7 +44,7 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 
 MyData rec1 = (MyData) 
mock.getReceivedExchanges().get(0).getIn().getBody(List.class).get(0);
 MyData rec2 = (MyData) 
mock.getReceivedExchanges().get(0).getIn().getBody(List.class).get(1);
-   
+
 //MyData rec1 = (MyData) map1.values().iterator().next();
 //MyData rec2 = (MyData) map2.values().iterator().next();
 
@@ -58,7 +61,8 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 public void testBindyPipeDelimiterMarshal() throws Exception {
 MockEndpoint mock = getMockEndpoint("mock:result");
 mock.expectedMessageCount(1);
-mock.message(0).body().startsWith("HAPPY|NEW|YEAR");
+mock.message(0).body().startsWith("col1|col2|col3");
+//mock.message(0).body().("HAPPY|NEW|YEAR");
 
 MyData data = new MyData();
 data.setCol1("HAPPY");
@@ -67,6 +71,7 @@ public class BindyPipeDelimiterTest extends CamelTestSupport {
 template.sendBody("direct:marshal", data);
 
 assertMockEndpointsSatisfied();
+System.out.println(mock.message(0).body().convertToString());
 }
 
 @Override
@@ -75,12 +80,14 @@ public class BindyPipeDelimiterTest extends 
CamelTestSupport {
 @Override
 public void configure() throws Exception {
 from("direct:unmarshal")
-.unmarshal().bindy(BindyType.Csv, 
org.apache.camel.dataformat.bindy.model.simple.pipeline.MyData.class)
-.to("mock:result");
+.unmarshal().

camel git commit: CAMEL-10103: More unittest

2016-07-01 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 7fb7e8f7a -> 158b8bb68


CAMEL-10103: More unittest


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/158b8bb6
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/158b8bb6
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/158b8bb6

Branch: refs/heads/master
Commit: 158b8bb68efb9b885a296b78e0218218dfda92ee
Parents: 7fb7e8f
Author: Arno Noordover 
Authored: Sat Jul 2 01:01:16 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jul 2 01:01:16 2016 +0200

--
 .../file/remote/OsgiParserFactoryTest.java  | 55 +++-
 1 file changed, 54 insertions(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/158b8bb6/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
--
diff --git 
a/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
 
b/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
index 10f1719..785c224 100644
--- 
a/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
+++ 
b/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
@@ -19,7 +19,12 @@ package org.apache.camel.component.file.remote;
 import org.apache.commons.net.ftp.FTPClientConfig;
 import org.apache.commons.net.ftp.FTPFileEntryParser;
 import org.apache.commons.net.ftp.parser.CompositeFileEntryParser;
+import org.apache.commons.net.ftp.parser.MVSFTPEntryParser;
+import org.apache.commons.net.ftp.parser.MacOsPeterFTPEntryParser;
 import org.apache.commons.net.ftp.parser.NTFTPEntryParser;
+import org.apache.commons.net.ftp.parser.NetwareFTPEntryParser;
+import org.apache.commons.net.ftp.parser.OS2FTPEntryParser;
+import org.apache.commons.net.ftp.parser.OS400FTPEntryParser;
 import org.apache.commons.net.ftp.parser.UnixFTPEntryParser;
 import org.apache.commons.net.ftp.parser.VMSVersioningFTPEntryParser;
 import org.junit.Before;
@@ -96,9 +101,57 @@ public class OsgiParserFactoryTest {
 @Test
 public void createFileEntryParserWin32()
 throws Exception {
-when(ftpClientConfig.getServerSystemKey()).thenReturn("WIN32");
+when(ftpClientConfig.getServerSystemKey()).thenReturn("bla WIN32 bla");
 FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
 assertThat(result, instanceOf(CompositeFileEntryParser.class));
 }
 
+@Test
+public void createFileEntryParserOs2()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("bla OS/2 bla");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(OS2FTPEntryParser.class));
+}
+
+@Test
+public void createFileEntryParserPlainOs400()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("OS/400");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(OS400FTPEntryParser.class));
+}
+
+@Test
+public void createFileEntryParserNotPlainOs400()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("OS/400 bla");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(CompositeFileEntryParser.class));
+}
+
+@Test
+public void createFileEntryParserMvs()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("bla MvS bla");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(MVSFTPEntryParser.class));
+}
+
+@Test
+public void createFileEntryParserNetware()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("bla NeTwArE 
bla");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(NetwareFTPEntryParser.class));
+}
+
+@Test
+public void createFileEntryParserMacOsPeter()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("bla MaCoS PeTER 
bla");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(MacOsPeterFTPEntryParser.class));
+}
+
 }
\ No newline at end of file



camel git commit: CAMEL-10103: Camel FTP - More unittest on OSGI Parsers

2016-07-01 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master f0a7e744e -> ca660ed24


CAMEL-10103: Camel FTP - More unittest on OSGI Parsers


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/ca660ed2
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/ca660ed2
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/ca660ed2

Branch: refs/heads/master
Commit: ca660ed249429b097b734a9e968c89fc4ff27906
Parents: f0a7e74
Author: Arno Noordover 
Authored: Fri Jul 1 13:02:35 2016 +0200
Committer: Arno Noordover 
Committed: Fri Jul 1 13:02:35 2016 +0200

--
 .../camel/component/file/remote/OsgiParserFactoryTest.java   | 8 
 1 file changed, 8 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/ca660ed2/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
--
diff --git 
a/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
 
b/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
index e5702be..10f1719 100644
--- 
a/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
+++ 
b/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
@@ -93,4 +93,12 @@ public class OsgiParserFactoryTest {
 assertThat(result, instanceOf(CompositeFileEntryParser.class));
 }
 
+@Test
+public void createFileEntryParserWin32()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("WIN32");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(CompositeFileEntryParser.class));
+}
+
 }
\ No newline at end of file



camel git commit: CAMEL-10103: Camel FTP - Unittest on OSGI Parsers

2016-06-30 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 02ace89eb -> eb7e8244c


CAMEL-10103: Camel FTP - Unittest on OSGI Parsers


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/eb7e8244
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/eb7e8244
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/eb7e8244

Branch: refs/heads/master
Commit: eb7e8244c22f7cb81ec191a218ca3d6a856ea329
Parents: 02ace89
Author: Arno Noordover 
Authored: Thu Jun 30 22:40:55 2016 +0200
Committer: Arno Noordover 
Committed: Thu Jun 30 22:40:55 2016 +0200

--
 components/camel-ftp/pom.xml|  5 +
 .../file/remote/OsgiParserFactoryTest.java  | 96 
 2 files changed, 101 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/eb7e8244/components/camel-ftp/pom.xml
--
diff --git a/components/camel-ftp/pom.xml b/components/camel-ftp/pom.xml
index 6e4691e..1b36540 100644
--- a/components/camel-ftp/pom.xml
+++ b/components/camel-ftp/pom.xml
@@ -128,6 +128,11 @@
   test
 
 
+
+  org.mockito
+  mockito-core
+  test
+
   
 
   

http://git-wip-us.apache.org/repos/asf/camel/blob/eb7e8244/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
--
diff --git 
a/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
 
b/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
new file mode 100644
index 000..e5702be
--- /dev/null
+++ 
b/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/OsgiParserFactoryTest.java
@@ -0,0 +1,96 @@
+/**
+ * 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.file.remote;
+
+import org.apache.commons.net.ftp.FTPClientConfig;
+import org.apache.commons.net.ftp.FTPFileEntryParser;
+import org.apache.commons.net.ftp.parser.CompositeFileEntryParser;
+import org.apache.commons.net.ftp.parser.NTFTPEntryParser;
+import org.apache.commons.net.ftp.parser.UnixFTPEntryParser;
+import org.apache.commons.net.ftp.parser.VMSVersioningFTPEntryParser;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mock;
+import org.mockito.Mockito;
+import org.mockito.runners.MockitoJUnitRunner;
+
+import static org.hamcrest.core.IsInstanceOf.instanceOf;
+import static org.junit.Assert.assertThat;
+import static org.mockito.Mockito.when;
+
+@RunWith(MockitoJUnitRunner.class)
+public class OsgiParserFactoryTest {
+
+private static final OsgiParserFactory OSGI_PARSER_FACTORY = new 
OsgiParserFactory(null);
+@Mock
+private FTPClientConfig ftpClientConfig;
+
+@Before
+public void setup() {
+
when(ftpClientConfig.getDefaultDateFormatStr()).thenReturn("-MM-dd");
+}
+
+@Test
+public void createFileEntryParserUnix()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("bla unix bla");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(UnixFTPEntryParser.class));
+}
+
+@Test
+public void createFileEntryParserLinux()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("bla linux bla");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(UnixFTPEntryParser.class));
+}
+
+@Test
+public void createFileEntryParserTypeL8()
+throws Exception {
+when(ftpClientConfig.getServerSystemKey()).thenReturn("bla type: l8 
bla");
+FTPFileEntryParser result = 
OSGI_PARSER_FACTORY.createFileEntryParser(ftpClientConfig);
+assertThat(result, instanceOf(UnixFTPEntryParser.class));
+}
+
+@Test
+

camel git commit: CAMEL-10103: Camel FTP - Unknown parser type

2016-06-29 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/camel-2.16.x 2ad57ce0d -> 6cb2fb254


CAMEL-10103: Camel FTP - Unknown parser type


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/6cb2fb25
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/6cb2fb25
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/6cb2fb25

Branch: refs/heads/camel-2.16.x
Commit: 6cb2fb25469f17cfa95a033362b74c97c5bc1ed3
Parents: 2ad57ce
Author: Arno Noordover 
Authored: Wed Jun 29 23:25:53 2016 +0200
Committer: Arno Noordover 
Committed: Thu Jun 30 06:20:00 2016 +0200

--
 .../file/remote/OsgiParserFactory.java  | 26 +++-
 1 file changed, 14 insertions(+), 12 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/6cb2fb25/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
--
diff --git 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
index 6eaf87d..9b7db4d 100644
--- 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
+++ 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
@@ -94,31 +94,33 @@ public class OsgiParserFactory extends 
DefaultFTPFileEntryParserFactory {
 throw new ParserInitializationException(parserClass.getName()
 + " does not implement the interface "
 + "org.apache.commons.net.ftp.FTPFileEntryParser.", e);
-} catch (Exception e) {
-throw new ParserInitializationException("Error initializing 
parser", e);
-} catch (ExceptionInInitializerError e) {
+} catch (Exception | ExceptionInInitializerError e) {
 throw new ParserInitializationException("Error initializing 
parser", e);
 }
 }
 if (parser == null) {
 String ukey = key.toUpperCase(Locale.ENGLISH);
-if (ukey.indexOf("UNIX") >= 0) {
+if (ukey.contains("UNIX")) {
+parser = new UnixFTPEntryParser(config);
+} else if (ukey.contains("LINUX")) {
 parser = new UnixFTPEntryParser(config);
-} else if (ukey.indexOf("VMS") >= 0) {
+} else if (ukey.contains("VMS")) {
 parser = new VMSVersioningFTPEntryParser(config);
-} else if (ukey.indexOf("WINDOWS") >= 0) {
+} else if (ukey.contains("WINDOWS")) {
+parser = createNTFTPEntryParser(config);
+} else if (ukey.contains("WIN32")) {
 parser = createNTFTPEntryParser(config);
-} else if (ukey.indexOf("OS/2") >= 0) {
+} else if (ukey.contains("OS/2")) {
 parser = new OS2FTPEntryParser(config);
-} else if ((ukey.indexOf("OS/400") >= 0) || 
(ukey.indexOf("AS/400") >= 0)) {
+} else if ((ukey.contains("OS/400")) || (ukey.contains("AS/400"))) 
{
 parser = createOS400FTPEntryParser(config);
-} else if (ukey.indexOf("MVS") >= 0) {
+} else if (ukey.contains("MVS")) {
 parser = new MVSFTPEntryParser();
-} else if (ukey.indexOf("NETWARE") >= 0) {
+} else if (ukey.contains("NETWARE")) {
 parser = new NetwareFTPEntryParser(config);
-} else if (ukey.indexOf("MACOS PETER") >= 0) {
+} else if (ukey.contains("MACOS PETER")) {
 parser = new MacOsPeterFTPEntryParser(config);
-} else if (ukey.indexOf("TYPE: L8") >= 0) {
+} else if (ukey.contains("TYPE: L8")) {
 parser = new UnixFTPEntryParser(config);
 } else {
 throw new ParserInitializationException("Unknown parser type: 
" + key);



camel git commit: CAMEL-10103: Camel FTP - Unknown parser type

2016-06-29 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/camel-2.17.x e2f31fa0a -> 380cd6489


CAMEL-10103: Camel FTP - Unknown parser type


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/380cd648
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/380cd648
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/380cd648

Branch: refs/heads/camel-2.17.x
Commit: 380cd648902425f6abea92f25285c1a65122182a
Parents: e2f31fa
Author: Arno Noordover 
Authored: Wed Jun 29 23:25:53 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 29 23:33:54 2016 +0200

--
 .../file/remote/OsgiParserFactory.java  | 26 +++-
 1 file changed, 14 insertions(+), 12 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/380cd648/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
--
diff --git 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
index 7d78ac1..b716a62 100644
--- 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
+++ 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
@@ -93,31 +93,33 @@ public class OsgiParserFactory extends 
DefaultFTPFileEntryParserFactory {
 throw new ParserInitializationException(parserClass.getName()
 + " does not implement the interface "
 + "org.apache.commons.net.ftp.FTPFileEntryParser.", e);
-} catch (Exception e) {
-throw new ParserInitializationException("Error initializing 
parser", e);
-} catch (ExceptionInInitializerError e) {
+} catch (Exception | ExceptionInInitializerError e) {
 throw new ParserInitializationException("Error initializing 
parser", e);
 }
 }
 if (parser == null) {
 String ukey = key.toUpperCase(Locale.ENGLISH);
-if (ukey.indexOf("UNIX") >= 0) {
+if (ukey.contains("UNIX")) {
+parser = new UnixFTPEntryParser(config);
+} else if (ukey.contains("LINUX")) {
 parser = new UnixFTPEntryParser(config);
-} else if (ukey.indexOf("VMS") >= 0) {
+} else if (ukey.contains("VMS")) {
 parser = new VMSVersioningFTPEntryParser(config);
-} else if (ukey.indexOf("WINDOWS") >= 0) {
+} else if (ukey.contains("WINDOWS")) {
+parser = createNTFTPEntryParser(config);
+} else if (ukey.contains("WIN32")) {
 parser = createNTFTPEntryParser(config);
-} else if (ukey.indexOf("OS/2") >= 0) {
+} else if (ukey.contains("OS/2")) {
 parser = new OS2FTPEntryParser(config);
-} else if ((ukey.indexOf("OS/400") >= 0) || 
(ukey.indexOf("AS/400") >= 0)) {
+} else if ((ukey.contains("OS/400")) || (ukey.contains("AS/400"))) 
{
 parser = createOS400FTPEntryParser(config);
-} else if (ukey.indexOf("MVS") >= 0) {
+} else if (ukey.contains("MVS")) {
 parser = new MVSFTPEntryParser();
-} else if (ukey.indexOf("NETWARE") >= 0) {
+} else if (ukey.contains("NETWARE")) {
 parser = new NetwareFTPEntryParser(config);
-} else if (ukey.indexOf("MACOS PETER") >= 0) {
+} else if (ukey.contains("MACOS PETER")) {
 parser = new MacOsPeterFTPEntryParser(config);
-} else if (ukey.indexOf("TYPE: L8") >= 0) {
+} else if (ukey.contains("TYPE: L8")) {
 parser = new UnixFTPEntryParser(config);
 } else {
 throw new ParserInitializationException("Unknown parser type: 
" + key);



camel git commit: CAMEL-10103: Camel FTP - Unknown parser type

2016-06-29 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master e92d754ed -> 65534ed75


CAMEL-10103: Camel FTP - Unknown parser type


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/65534ed7
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/65534ed7
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/65534ed7

Branch: refs/heads/master
Commit: 65534ed757e54f31ccf38e82f5fa56a19ed653f3
Parents: e92d754
Author: Arno Noordover 
Authored: Wed Jun 29 23:25:53 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 29 23:25:53 2016 +0200

--
 .../file/remote/OsgiParserFactory.java  | 26 +++-
 1 file changed, 14 insertions(+), 12 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/65534ed7/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
--
diff --git 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
index 9c4875e..dbb0281 100644
--- 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
+++ 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/OsgiParserFactory.java
@@ -93,31 +93,33 @@ public class OsgiParserFactory extends 
DefaultFTPFileEntryParserFactory {
 throw new ParserInitializationException(parserClass.getName()
 + " does not implement the interface "
 + "org.apache.commons.net.ftp.FTPFileEntryParser.", e);
-} catch (Exception e) {
-throw new ParserInitializationException("Error initializing 
parser", e);
-} catch (ExceptionInInitializerError e) {
+} catch (Exception | ExceptionInInitializerError e) {
 throw new ParserInitializationException("Error initializing 
parser", e);
 }
 }
 if (parser == null) {
 String ukey = key.toUpperCase(Locale.ENGLISH);
-if (ukey.indexOf("UNIX") >= 0) {
+if (ukey.contains("UNIX")) {
+parser = new UnixFTPEntryParser(config);
+} else if (ukey.contains("LINUX")) {
 parser = new UnixFTPEntryParser(config);
-} else if (ukey.indexOf("VMS") >= 0) {
+} else if (ukey.contains("VMS")) {
 parser = new VMSVersioningFTPEntryParser(config);
-} else if (ukey.indexOf("WINDOWS") >= 0) {
+} else if (ukey.contains("WINDOWS")) {
+parser = createNTFTPEntryParser(config);
+} else if (ukey.contains("WIN32")) {
 parser = createNTFTPEntryParser(config);
-} else if (ukey.indexOf("OS/2") >= 0) {
+} else if (ukey.contains("OS/2")) {
 parser = new OS2FTPEntryParser(config);
-} else if ((ukey.indexOf("OS/400") >= 0) || 
(ukey.indexOf("AS/400") >= 0)) {
+} else if ((ukey.contains("OS/400")) || (ukey.contains("AS/400"))) 
{
 parser = createOS400FTPEntryParser(config);
-} else if (ukey.indexOf("MVS") >= 0) {
+} else if (ukey.contains("MVS")) {
 parser = new MVSFTPEntryParser();
-} else if (ukey.indexOf("NETWARE") >= 0) {
+} else if (ukey.contains("NETWARE")) {
 parser = new NetwareFTPEntryParser(config);
-} else if (ukey.indexOf("MACOS PETER") >= 0) {
+} else if (ukey.contains("MACOS PETER")) {
 parser = new MacOsPeterFTPEntryParser(config);
-} else if (ukey.indexOf("TYPE: L8") >= 0) {
+} else if (ukey.contains("TYPE: L8")) {
 parser = new UnixFTPEntryParser(config);
 } else {
 throw new ParserInitializationException("Unknown parser type: 
" + key);



camel git commit: CAMEL-9476: added marshall and unmarshall test for padding and trimming

2016-06-29 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 828ebfb09 -> e92d754ed


CAMEL-9476: added marshall and unmarshall test for padding and trimming


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/e92d754e
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/e92d754e
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/e92d754e

Branch: refs/heads/master
Commit: e92d754ed68065d3d8766b9bcfaf9bb6dda4f3f4
Parents: 828ebfb
Author: Arno Noordover 
Authored: Wed Jun 29 23:12:07 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 29 23:12:07 2016 +0200

--
 .../fixed/converter/BindyConverterTest.java | 128 +++
 1 file changed, 128 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/e92d754e/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/fixed/converter/BindyConverterTest.java
--
diff --git 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/fixed/converter/BindyConverterTest.java
 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/fixed/converter/BindyConverterTest.java
index 7ff80b3..2ac16e4 100644
--- 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/fixed/converter/BindyConverterTest.java
+++ 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/fixed/converter/BindyConverterTest.java
@@ -30,6 +30,7 @@ import 
org.apache.camel.dataformat.bindy.annotation.FixedLengthRecord;
 import org.apache.camel.model.dataformat.BindyDataFormat;
 import org.apache.camel.model.dataformat.BindyType;
 import org.apache.camel.test.junit4.CamelTestSupport;
+import org.hamcrest.core.Is;
 import org.junit.Assert;
 import org.junit.Test;
 import org.springframework.test.annotation.DirtiesContext;
@@ -40,8 +41,11 @@ public class BindyConverterTest extends CamelTestSupport {
 
 public static final String URI_DIRECT_MARSHALL = "direct:marshall";
 public static final String URI_DIRECT_UNMARSHALL   = 
"direct:unmarshall";
+public static final String URI_DIRECT_THROUGH = "direct:through";
+
 public static final String URI_MOCK_MARSHALL_RESULT= 
"mock:marshall-result";
 public static final String URI_MOCK_UNMARSHALL_RESULT  = 
"mock:unmarshall-result";
+public static final String URI_MOCK_THROUGH  = "mock:through-result";
 
 // 
*
 //
@@ -59,6 +63,12 @@ public class BindyConverterTest extends CamelTestSupport {
 @EndpointInject(uri = URI_MOCK_UNMARSHALL_RESULT)
 private MockEndpoint uresult;
 
+@Produce(uri = URI_DIRECT_THROUGH)
+private ProducerTemplate ttemplate;
+
+@EndpointInject(uri = URI_MOCK_THROUGH)
+private MockEndpoint tresult;
+
 // 
*
 // TEST
 // 
*
@@ -89,6 +99,89 @@ public class BindyConverterTest extends CamelTestSupport {
 Assert.assertEquals("0123456789", data.field1);
 }
 
+@Test
+@DirtiesContext
+public void testRightAlignedNotTrimmed() throws Exception {
+AllCombinations data = sendAndRecieveAllCombinations();
+
+assertThat("Right aligned, padding not trimmed", data.field1, 
Is.is("!!!f1"));
+}
+
+@Test
+@DirtiesContext
+public void testLeftAlignedNotTrimmed() throws Exception {
+AllCombinations data = sendAndRecieveAllCombinations();
+
+assertThat("Left aligned, padding not trimmed", data.field2, 
Is.is("f2!!!"));
+}
+
+@Test
+@DirtiesContext
+public void testRightAlignedTrimmed() throws Exception {
+AllCombinations data = sendAndRecieveAllCombinations();
+
+assertThat("Right aligned, padding trimmed", data.field3, Is.is("f3"));
+}
+
+@Test
+@DirtiesContext
+public void testLeftAlignedTrimmed() throws Exception {
+AllCombinations data = sendAndRecieveAllCombinations();
+
+assertThat("Left aligned, padding trimmed", data.field4, Is.is("f4"));
+}
+
+@Test
+@DirtiesContext
+public void testRightAlignedRecordPaddingNotTrimmed() throws Exception {
+AllCombinations data = sendAndRecieveAllCombinations();
+
+assertThat("Right aligned, padding not trimmed", data.field5, 
Is.is("###f5"));
+}
+
+@Test
+@DirtiesContext
+public void testLeftAlignedRecordPaddingNotTrimmed() throws Exception {
+AllCombinations data = sendAndRecieveAllCombinations();
+
+assertThat("Left aligned, padding not trimmed", data.field6, 
Is.is("f6###"));
+}
+
+@Test
+@DirtiesContext
+public void testRightAlignedRecor

[1/2] camel git commit: JSSEDocumentation

2016-06-29 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 6bb3988aa -> 828ebfb09


JSSEDocumentation


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/828ebfb0
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/828ebfb0
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/828ebfb0

Branch: refs/heads/master
Commit: 828ebfb0900c3eecce78097af63a191c9dc9eaa9
Parents: fdc742d
Author: Arno Noordover 
Authored: Wed Jun 29 20:54:28 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 29 20:59:04 2016 +0200

--
 .../en/camel-configuration-utilities.adoc   | 558 ++-
 1 file changed, 163 insertions(+), 395 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/828ebfb0/docs/user-manual/en/camel-configuration-utilities.adoc
--
diff --git a/docs/user-manual/en/camel-configuration-utilities.adoc 
b/docs/user-manual/en/camel-configuration-utilities.adoc
index 15e3689..24ba0de 100644
--- a/docs/user-manual/en/camel-configuration-utilities.adoc
+++ b/docs/user-manual/en/camel-configuration-utilities.adoc
@@ -70,18 +70,40 @@ namespaces for Spring and Blueprint based configuration, 
respectively.
 SSLContextParameters
 
 
-
-[cols="a,a",width="100%",options="header",grids="cols"]
-|
-|Property|Description
-|Field: +
-cipherSuites +
-Class: +
-CipherSuitesParameters +
-XML: +
-sslContextParameters +
-/ciphersuites
-|This optional property represents a collection of explicitly named
+SSLContextParameters (sslContextParameters) contain the following elements:
+
+* provider (attribute)
+* secureSocketProtocol (attribute)
+* certAlias (attribute)
+* sessionTimeout (attribute)
+* cipherSuites (element)
+* cipherSuitesFilter (element)
+* secureSocketProtocols (element)
+* secureSocketProtocolsFilter (element)
+* keyManagers (element)
+* trustManagers (element)
+* secureRandom (element)
+* clientParameters (element)
+* serverParameters (elment)
+
+provider::
+The optional provider identifier for the JSSE implementation to use when
+constructing the SSLContext.  If omitted, the standard provider look-up
+mechanism is used to resolve the provider.
+secureSocketProtocol::
+The optional secure socket protocol. See 
http://download.oracle.com/javase/6/docs/technotes/guides//security/jsse/JSSERefGuide.html#AppA[Appendix
+A] in the Java Secure Socket Extension Reference Guide for information
+about standard protocol names.  If omitted, TLS is used by default. 
+Note that this property is related to but distinctly different from the
+secureSocketProtocols and secureSocketProtocolsFilter properties.
+certAlias::
+*Camel 2.13:* An optional certificate alias to use. This is useful when
+the keystore has multiple certificates.
+sessionTimeout::
+This optional property defines the timeout period, in seconds, for
+sessions on both the client and server side as well as in the SSLEngine.
+cipherSuites::
+This optional property represents a collection of explicitly named
 cipher suites to enable on both the client and server side as well as in
 the SSLEngine.  These values take precedence over filters supplied in
 cipherSuitesFilter.  The utility attempts to enable the listed cipher
@@ -89,37 +111,17 @@ suites regardless of whether or not the JSSE provider 
actually supports
 them or not.  This behavior guarantees that listed cipher suites are
 always enabled when listed.  For a more lenient option, use
 cipherSuitesFilter.
-|Field: +
-cipherSuitesFilter +
-Class: +
-link:camel-configuration-utilities.html[FilterParameters] +
-XML: +
-sslContextParameters +
-/cipherSuitesFilter
-|This optional property represents a collection of include and exclude
+cipherSuitesFilter::
+This optional property represents a collection of include and exclude
 patterns for cipher suites to enable on both the client and server side
 as well as in the SSLEngine.  The patterns are applied over only the
 available cipher suites.  The exclude patterns have precedence over the
 include patterns.  If no cipherSuites and no cipherSuitesFilter are
 present, the default patterns applied are: +
-Includes
-
-* .*
-
-Excludes
-
-* .*_NULL_.*
-* .*_anon_.*
-* .*DES.* *Camel 2.15.4*
-* .*EXPORT.* *Camel 2.15.4*
-|Field: +
-secureSocketProtocols +
-Class: +
-SecureSocketProtocolsParameters +
-XML: +
-sslContextParameters +
-/secureSocketProtocols
-|This optional property represents a collection of explicitly named
+Includes .\*; Excludes .*_NULL_.*, .\*_anon_.*
+, .\*DES.* *Camel 2.15.4*, .\*EXPORT.* *Camel 2.15.4*.
+secureSocketProtocols::
+This optional property represents a collection of explicitly named
 secure socket protocols, such as SSLv3/TLS/etc., to enable on both the
 client and server side as well as in the SSL

[2/2] camel git commit: JSSEDocumentation

2016-06-29 Thread anoordover
JSSEDocumentation


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/fdc742df
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/fdc742df
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/fdc742df

Branch: refs/heads/master
Commit: fdc742dfe41200af301a31247e81c0bfde209f1a
Parents: 6bb3988
Author: Arno Noordover 
Authored: Mon Jun 27 19:21:54 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 29 20:59:04 2016 +0200

--
 docs/user-manual/en/SUMMARY.md  |   1 +
 .../en/camel-configuration-utilities.adoc   | 844 +++
 2 files changed, 845 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/fdc742df/docs/user-manual/en/SUMMARY.md
--
diff --git a/docs/user-manual/en/SUMMARY.md b/docs/user-manual/en/SUMMARY.md
index 1c4ef4a..bfce4c7 100644
--- a/docs/user-manual/en/SUMMARY.md
+++ b/docs/user-manual/en/SUMMARY.md
@@ -344,3 +344,4 @@
 * [Blueprint Testing](test-blueprint.adoc)
 * [CDI Testing](cdi-testing.adoc)
 * [Spring Testing](spring-testing.adoc)
+* [JSSE Utility](camel-configuration-utilities.adoc)

http://git-wip-us.apache.org/repos/asf/camel/blob/fdc742df/docs/user-manual/en/camel-configuration-utilities.adoc
--
diff --git a/docs/user-manual/en/camel-configuration-utilities.adoc 
b/docs/user-manual/en/camel-configuration-utilities.adoc
new file mode 100644
index 000..15e3689
--- /dev/null
+++ b/docs/user-manual/en/camel-configuration-utilities.adoc
@@ -0,0 +1,844 @@
+[[CamelConfigurationUtilities-JSSEUtility]]
+JSSE Utility
+
+
+The JSSE Utility, available as of *2.8*, allows you to easily configure
+aspects of the
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html[Java
+Secure Socket Extension] (JSSE) API in order to greatly simplify the use
+of custom transport layer security (TLS) settings on Camel components.
+
+[[CamelConfigurationUtilities-SupportedComponents]]
+Supported Components
+
+
+The following Camel components directly support the use of this
+configuration utility:
+
+* link:http4.html[HTTP4]
+* link:jetty.html[Jetty]
+* link:ahc.html[AHC]
+* link:netty.html[Netty]
+* link:cometd.html[Cometd]
+* link:ftp2.html[FTP2]
+* link:irc.html[IRC]
+* link:mail.html[Mail]
+* MINA 2
+
+The following Camel components indirectly support the use of this
+configuration utility:
+
+* link:cxf.html[CXF]
+* link:http.html[HTTP]
+
+[[CamelConfigurationUtilities-Configuration]]
+Configuration
+^
+
+The key component in configuring TLS through the JSSE API is the
+SSLContext.  The
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html#SSLContext[SSLContext]
+provides socket factories for both
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html#SSLSocketFactory[client-side]
+and
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html#SSLSocketFactory[server-side]
+sockets as well as another component called an
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html#SSLENG[SSLEngine]
+that is used by non-blocking IO to support TLS.  The JSSE configuration
+utility provides an easy to use builder for configuring these JSSE
+components, among others, in a manner that allows you to provide all
+configuration options up front during the initialization of your
+application such that you don't have to customize library code or dig
+though the inner workings of a third-party library in order to inject
+hooks for the configuration of each component in the JSSE API.  The
+central builder in the JSSE configuration utility is the
+SSLContextParameters.  This class serves as the entry point for most
+configuration in the JSSE utility.
+
+[NOTE]
+=
+All non-native classes are in the org.apache.camel.util.jsse package. 
+All non-W3C schema defined types are in the
+http://camel.apache.org/schema/spring[http://camel.apache.org/schema/spring]
+or
+http://camel.apache.org/schema/blueprint[http://camel.apache.org/schema/blueprint]
+namespaces for Spring and Blueprint based configuration, respectively.
+=
+
+
+[[CamelConfigurationUtilities-SSLContextParameters]]
+SSLContextParameters
+
+
+
+[cols="a,a",width="100%",options="header",grids="cols"]
+|
+|Property|Description
+|Field: +
+cipherSuites +
+Class: +
+CipherSuitesParameters +
+XML: +
+sslContextParameters +
+/ciphersuites
+|This optional property represents a collection of explicitly named
+cipher suites

[camel] Git Push Summary

2016-06-29 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/JSSEDocumentation [deleted] 828ebfb09


[2/2] camel git commit: JSSEDocumentation

2016-06-29 Thread anoordover
JSSEDocumentation


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/fdc742df
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/fdc742df
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/fdc742df

Branch: refs/heads/JSSEDocumentation
Commit: fdc742dfe41200af301a31247e81c0bfde209f1a
Parents: 6bb3988
Author: Arno Noordover 
Authored: Mon Jun 27 19:21:54 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 29 20:59:04 2016 +0200

--
 docs/user-manual/en/SUMMARY.md  |   1 +
 .../en/camel-configuration-utilities.adoc   | 844 +++
 2 files changed, 845 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/fdc742df/docs/user-manual/en/SUMMARY.md
--
diff --git a/docs/user-manual/en/SUMMARY.md b/docs/user-manual/en/SUMMARY.md
index 1c4ef4a..bfce4c7 100644
--- a/docs/user-manual/en/SUMMARY.md
+++ b/docs/user-manual/en/SUMMARY.md
@@ -344,3 +344,4 @@
 * [Blueprint Testing](test-blueprint.adoc)
 * [CDI Testing](cdi-testing.adoc)
 * [Spring Testing](spring-testing.adoc)
+* [JSSE Utility](camel-configuration-utilities.adoc)

http://git-wip-us.apache.org/repos/asf/camel/blob/fdc742df/docs/user-manual/en/camel-configuration-utilities.adoc
--
diff --git a/docs/user-manual/en/camel-configuration-utilities.adoc 
b/docs/user-manual/en/camel-configuration-utilities.adoc
new file mode 100644
index 000..15e3689
--- /dev/null
+++ b/docs/user-manual/en/camel-configuration-utilities.adoc
@@ -0,0 +1,844 @@
+[[CamelConfigurationUtilities-JSSEUtility]]
+JSSE Utility
+
+
+The JSSE Utility, available as of *2.8*, allows you to easily configure
+aspects of the
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html[Java
+Secure Socket Extension] (JSSE) API in order to greatly simplify the use
+of custom transport layer security (TLS) settings on Camel components.
+
+[[CamelConfigurationUtilities-SupportedComponents]]
+Supported Components
+
+
+The following Camel components directly support the use of this
+configuration utility:
+
+* link:http4.html[HTTP4]
+* link:jetty.html[Jetty]
+* link:ahc.html[AHC]
+* link:netty.html[Netty]
+* link:cometd.html[Cometd]
+* link:ftp2.html[FTP2]
+* link:irc.html[IRC]
+* link:mail.html[Mail]
+* MINA 2
+
+The following Camel components indirectly support the use of this
+configuration utility:
+
+* link:cxf.html[CXF]
+* link:http.html[HTTP]
+
+[[CamelConfigurationUtilities-Configuration]]
+Configuration
+^
+
+The key component in configuring TLS through the JSSE API is the
+SSLContext.  The
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html#SSLContext[SSLContext]
+provides socket factories for both
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html#SSLSocketFactory[client-side]
+and
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html#SSLSocketFactory[server-side]
+sockets as well as another component called an
+http://download.oracle.com/javase/6/docs/technotes/guides/security/jsse/JSSERefGuide.html#SSLENG[SSLEngine]
+that is used by non-blocking IO to support TLS.  The JSSE configuration
+utility provides an easy to use builder for configuring these JSSE
+components, among others, in a manner that allows you to provide all
+configuration options up front during the initialization of your
+application such that you don't have to customize library code or dig
+though the inner workings of a third-party library in order to inject
+hooks for the configuration of each component in the JSSE API.  The
+central builder in the JSSE configuration utility is the
+SSLContextParameters.  This class serves as the entry point for most
+configuration in the JSSE utility.
+
+[NOTE]
+=
+All non-native classes are in the org.apache.camel.util.jsse package. 
+All non-W3C schema defined types are in the
+http://camel.apache.org/schema/spring[http://camel.apache.org/schema/spring]
+or
+http://camel.apache.org/schema/blueprint[http://camel.apache.org/schema/blueprint]
+namespaces for Spring and Blueprint based configuration, respectively.
+=
+
+
+[[CamelConfigurationUtilities-SSLContextParameters]]
+SSLContextParameters
+
+
+
+[cols="a,a",width="100%",options="header",grids="cols"]
+|
+|Property|Description
+|Field: +
+cipherSuites +
+Class: +
+CipherSuitesParameters +
+XML: +
+sslContextParameters +
+/ciphersuites
+|This optional property represents a collection of explicitly named
+ci

[1/2] camel git commit: JSSEDocumentation

2016-06-29 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/JSSEDocumentation [created] 828ebfb09


JSSEDocumentation


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/828ebfb0
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/828ebfb0
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/828ebfb0

Branch: refs/heads/JSSEDocumentation
Commit: 828ebfb0900c3eecce78097af63a191c9dc9eaa9
Parents: fdc742d
Author: Arno Noordover 
Authored: Wed Jun 29 20:54:28 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 29 20:59:04 2016 +0200

--
 .../en/camel-configuration-utilities.adoc   | 558 ++-
 1 file changed, 163 insertions(+), 395 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/828ebfb0/docs/user-manual/en/camel-configuration-utilities.adoc
--
diff --git a/docs/user-manual/en/camel-configuration-utilities.adoc 
b/docs/user-manual/en/camel-configuration-utilities.adoc
index 15e3689..24ba0de 100644
--- a/docs/user-manual/en/camel-configuration-utilities.adoc
+++ b/docs/user-manual/en/camel-configuration-utilities.adoc
@@ -70,18 +70,40 @@ namespaces for Spring and Blueprint based configuration, 
respectively.
 SSLContextParameters
 
 
-
-[cols="a,a",width="100%",options="header",grids="cols"]
-|
-|Property|Description
-|Field: +
-cipherSuites +
-Class: +
-CipherSuitesParameters +
-XML: +
-sslContextParameters +
-/ciphersuites
-|This optional property represents a collection of explicitly named
+SSLContextParameters (sslContextParameters) contain the following elements:
+
+* provider (attribute)
+* secureSocketProtocol (attribute)
+* certAlias (attribute)
+* sessionTimeout (attribute)
+* cipherSuites (element)
+* cipherSuitesFilter (element)
+* secureSocketProtocols (element)
+* secureSocketProtocolsFilter (element)
+* keyManagers (element)
+* trustManagers (element)
+* secureRandom (element)
+* clientParameters (element)
+* serverParameters (elment)
+
+provider::
+The optional provider identifier for the JSSE implementation to use when
+constructing the SSLContext.  If omitted, the standard provider look-up
+mechanism is used to resolve the provider.
+secureSocketProtocol::
+The optional secure socket protocol. See 
http://download.oracle.com/javase/6/docs/technotes/guides//security/jsse/JSSERefGuide.html#AppA[Appendix
+A] in the Java Secure Socket Extension Reference Guide for information
+about standard protocol names.  If omitted, TLS is used by default. 
+Note that this property is related to but distinctly different from the
+secureSocketProtocols and secureSocketProtocolsFilter properties.
+certAlias::
+*Camel 2.13:* An optional certificate alias to use. This is useful when
+the keystore has multiple certificates.
+sessionTimeout::
+This optional property defines the timeout period, in seconds, for
+sessions on both the client and server side as well as in the SSLEngine.
+cipherSuites::
+This optional property represents a collection of explicitly named
 cipher suites to enable on both the client and server side as well as in
 the SSLEngine.  These values take precedence over filters supplied in
 cipherSuitesFilter.  The utility attempts to enable the listed cipher
@@ -89,37 +111,17 @@ suites regardless of whether or not the JSSE provider 
actually supports
 them or not.  This behavior guarantees that listed cipher suites are
 always enabled when listed.  For a more lenient option, use
 cipherSuitesFilter.
-|Field: +
-cipherSuitesFilter +
-Class: +
-link:camel-configuration-utilities.html[FilterParameters] +
-XML: +
-sslContextParameters +
-/cipherSuitesFilter
-|This optional property represents a collection of include and exclude
+cipherSuitesFilter::
+This optional property represents a collection of include and exclude
 patterns for cipher suites to enable on both the client and server side
 as well as in the SSLEngine.  The patterns are applied over only the
 available cipher suites.  The exclude patterns have precedence over the
 include patterns.  If no cipherSuites and no cipherSuitesFilter are
 present, the default patterns applied are: +
-Includes
-
-* .*
-
-Excludes
-
-* .*_NULL_.*
-* .*_anon_.*
-* .*DES.* *Camel 2.15.4*
-* .*EXPORT.* *Camel 2.15.4*
-|Field: +
-secureSocketProtocols +
-Class: +
-SecureSocketProtocolsParameters +
-XML: +
-sslContextParameters +
-/secureSocketProtocols
-|This optional property represents a collection of explicitly named
+Includes .\*; Excludes .*_NULL_.*, .\*_anon_.*
+, .\*DES.* *Camel 2.15.4*, .\*EXPORT.* *Camel 2.15.4*.
+secureSocketProtocols::
+This optional property represents a collection of explicitly named
 secure socket protocols, such as SSLv3/TLS/etc., to enable on both the
 client and server side as

camel git commit: CAMEL-10090: fixed failing test on https://builds.apache.org/view/All/job/Camel.trunk.fulltest.java8/

2016-06-28 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 0e72f475f -> 13b39177c


CAMEL-10090: fixed failing test on 
https://builds.apache.org/view/All/job/Camel.trunk.fulltest.java8/


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/13b39177
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/13b39177
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/13b39177

Branch: refs/heads/master
Commit: 13b39177cfdb33123eaed49484ed3a53b4d17d37
Parents: 0e72f47
Author: Arno Noordover 
Authored: Tue Jun 28 23:35:03 2016 +0200
Committer: Arno Noordover 
Committed: Tue Jun 28 23:35:03 2016 +0200

--
 .../salesforce/internal/processor/JsonRestProcessorTest.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/13b39177/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessorTest.java
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessorTest.java
 
b/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessorTest.java
index caaa794..2d322aa 100644
--- 
a/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessorTest.java
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/test/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessorTest.java
@@ -55,7 +55,7 @@ public class JsonRestProcessorTest {
 exchange.getIn().setBody(doc);
 ByteArrayInputStream is = (ByteArrayInputStream) 
jsonRestProcessor.getRequestStream(exchange);
 String result = IOUtils.toString(is);
-assertThat(result, result.length(), Is.is(48));
+assertThat(result, result.length() <= 48, Is.is(true));
 }
 
 private static class TestObject extends AbstractDTOBase {



camel git commit: CAMEL-9476: added possibility to ignore too short lines for fixed layout

2016-06-28 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 05f13c00a -> 0e72f475f


CAMEL-9476: added possibility to ignore too short lines for fixed layout


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/0e72f475
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/0e72f475
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/0e72f475

Branch: refs/heads/master
Commit: 0e72f475f80a9699fe49004c431e1ce4e360c35d
Parents: 05f13c0
Author: Arno Noordover 
Authored: Tue Jun 28 22:00:39 2016 +0200
Committer: Arno Noordover 
Committed: Tue Jun 28 22:00:39 2016 +0200

--
 .../bindy/BindyFixedLengthFactory.java  | 61 +++-
 .../bindy/annotation/FixedLengthRecord.java |  4 +
 .../bindy/fixed/BindyFixedLengthDataFormat.java | 73 +--
 .../fixed/BindyPaddingAndTrimmingTest.java  | 97 
 ...impleFixedLengthUnmarshallTrimFieldTest.java |  6 +-
 5 files changed, 209 insertions(+), 32 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/0e72f475/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyFixedLengthFactory.java
--
diff --git 
a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyFixedLengthFactory.java
 
b/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyFixedLengthFactory.java
index c517852..302a2f7 100644
--- 
a/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyFixedLengthFactory.java
+++ 
b/components/camel-bindy/src/main/java/org/apache/camel/dataformat/bindy/BindyFixedLengthFactory.java
@@ -64,6 +64,7 @@ public class BindyFixedLengthFactory extends 
BindyAbstractFactory implements Bin
 private char paddingChar;
 private int recordLength;
 private boolean ignoreTrailingChars;
+private boolean ignoreMissingChars;
 
 private Class header;
 private Class footer;
@@ -201,7 +202,15 @@ public class BindyFixedLengthFactory extends 
BindyAbstractFactory implements Bin
 }
 
 if (length > 0) {
-token = record.substring(offset - 1, offset + length - 1);
+if (record.length() < offset) {
+token = "";
+} else {
+int endIndex = offset + length - 1;
+if (endIndex > record.length()) {
+endIndex = record.length();
+}
+token = record.substring(offset - 1, endIndex);
+}
 offset += length;
 } else if (!delimiter.equals("")) {
 String tempToken = record.substring(offset - 1, 
record.length());
@@ -214,7 +223,8 @@ public class BindyFixedLengthFactory extends 
BindyAbstractFactory implements Bin
 }
 
 if (dataField.trim()) {
-token = token.trim();
+token = trim(token, dataField, paddingChar);
+//token = token.trim();
 }
 
 // Check mandatory field
@@ -252,7 +262,10 @@ public class BindyFixedLengthFactory extends 
BindyAbstractFactory implements Bin
 // format the data received
 Object value = null;
 
-if (!token.equals("")) {
+if ("".equals(token)) {
+token = dataField.defaultValue();
+}
+if (!"".equals(token)) {
 try {
 value = format.parse(token);
 } catch (FormatException ie) {
@@ -287,6 +300,38 @@ public class BindyFixedLengthFactory extends 
BindyAbstractFactory implements Bin
 
 }
 
+private String trim(String token, DataField dataField, char paddingChar) {
+char myPaddingChar = dataField.paddingChar();
+if (dataField.paddingChar() == 0) {
+myPaddingChar = paddingChar;
+}
+if ("R".equals(dataField.align())) {
+return leftTrim(token, myPaddingChar);
+} else {
+return rightTrim(token, myPaddingChar);
+}
+}
+
+private String rightTrim(String token, char myPaddingChar) {
+StringBuilder sb = new StringBuilder(token);
+
+while (sb.length() > 0 && myPaddingChar == sb.charAt(sb.length() - 1)) 
{
+sb.deleteCharAt(sb.length() - 1);
+}
+
+return sb.toString();
+}
+
+private String leftTrim(String token, char myPaddingChar) {
+StringBuilder sb = new StringBuilder(token);
+
+while (sb.length() > 0 && myPaddingChar == (sb.charAt(0))) {
+sb.deleteCharAt(0);
+}
+
+return sb.toString();
+}
+
 @Override
 public String unbind(Map model) throws Exception {
 
@@ -516,6 +561,9 @@ public class BindyF

camel git commit: CAMEL-10090: CS fix

2016-06-27 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 0c1264921 -> 505bf099d


CAMEL-10090: CS fix


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/505bf099
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/505bf099
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/505bf099

Branch: refs/heads/master
Commit: 505bf099d96ec4d5a546f5f2ea38aab33a72cd88
Parents: 0c12649
Author: Arno Noordover 
Authored: Mon Jun 27 21:50:31 2016 +0200
Committer: Arno Noordover 
Committed: Mon Jun 27 21:50:31 2016 +0200

--
 .../salesforce/internal/processor/JsonRestProcessor.java| 9 -
 1 file changed, 9 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/505bf099/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessor.java
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessor.java
 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessor.java
index 1dad12c..53f31b3 100644
--- 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessor.java
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/processor/JsonRestProcessor.java
@@ -20,11 +20,6 @@ import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
-import java.text.DateFormat;
-import java.text.FieldPosition;
-import java.text.ParsePosition;
-import java.text.SimpleDateFormat;
-import java.util.Date;
 import java.util.List;
 
 import com.fasterxml.jackson.core.type.TypeReference;
@@ -46,10 +41,6 @@ import 
org.apache.camel.component.salesforce.api.dto.SearchResult;
 import org.apache.camel.component.salesforce.api.dto.Version;
 import org.apache.camel.component.salesforce.internal.joda.DateTimeModule;
 import org.eclipse.jetty.util.StringUtil;
-import org.joda.time.DateTime;
-import org.joda.time.format.DateTimeFormat;
-import org.joda.time.format.DateTimeFormatter;
-import org.joda.time.format.DateTimeFormatterBuilder;
 
 public class JsonRestProcessor extends AbstractRestProcessor {
 



camel git commit: CAMEL-10090: Salesforce should support jodatime

2016-06-27 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master bff52b4ae -> 0c1264921


CAMEL-10090: Salesforce should support jodatime


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/0c126492
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/0c126492
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/0c126492

Branch: refs/heads/master
Commit: 0c126492181bd4cfe936d01be4d8bd6f55062e3a
Parents: bff52b4
Author: Arno Noordover 
Authored: Sun Jun 26 19:15:53 2016 +0200
Committer: Arno Noordover 
Committed: Mon Jun 27 19:49:52 2016 +0200

--
 .../camel-salesforce-component/pom.xml  |   5 +
 .../salesforce/api/JodaTimeConverter.java   |  19 ++-
 .../internal/joda/DateTimeDeserializer.java |  64 
 .../internal/joda/DateTimeModule.java   |  31 
 .../internal/joda/DateTimeSerializer.java   |  60 
 .../internal/processor/JsonRestProcessor.java   |  11 ++
 .../salesforce/RestApiIntegrationTest.java  | 153 ---
 .../salesforce/dto/generated/Tasks__c.java  |  44 ++
 .../processor/JsonRestProcessorTest.java|  74 +
 9 files changed, 402 insertions(+), 59 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/0c126492/components/camel-salesforce/camel-salesforce-component/pom.xml
--
diff --git a/components/camel-salesforce/camel-salesforce-component/pom.xml 
b/components/camel-salesforce/camel-salesforce-component/pom.xml
index eac801c..af5507b 100644
--- a/components/camel-salesforce/camel-salesforce-component/pom.xml
+++ b/components/camel-salesforce/camel-salesforce-component/pom.xml
@@ -44,6 +44,11 @@
 
   
 
+  commons-io
+  commons-io
+  test
+
+
   org.apache.camel
   camel-core
 

http://git-wip-us.apache.org/repos/asf/camel/blob/0c126492/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/JodaTimeConverter.java
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/JodaTimeConverter.java
 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/JodaTimeConverter.java
index c2609b8..5cd19b1 100644
--- 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/JodaTimeConverter.java
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/api/JodaTimeConverter.java
@@ -28,14 +28,29 @@ import com.thoughtworks.xstream.io.HierarchicalStreamWriter;
 import org.joda.time.DateTime;
 import org.joda.time.DateTimeZone;
 import org.joda.time.format.DateTimeFormatter;
-import org.joda.time.format.ISODateTimeFormat;
+import org.joda.time.format.DateTimeFormatterBuilder;
 
 /**
  * XStream converter for handling JodaTime fields.
  */
 public class JodaTimeConverter implements Converter {
 
-private final DateTimeFormatter formatter = ISODateTimeFormat.dateTime();
+private final DateTimeFormatter formatter = new DateTimeFormatterBuilder()
+.appendYear(4, 4)
+.appendLiteral('-')
+.appendMonthOfYear(2)
+.appendLiteral('-')
+.appendDayOfMonth(2)
+.appendLiteral('T')
+.appendHourOfDay(2)
+.appendLiteral(':')
+.appendMinuteOfHour(2)
+.appendLiteral(':')
+.appendSecondOfMinute(2)
+.appendLiteral('.')
+.appendMillisOfSecond(3)
+.appendTimeZoneOffset("Z", true, 2, 2)
+.toFormatter();
 
 @Override
 public void marshal(Object o, HierarchicalStreamWriter writer, 
MarshallingContext context) {

http://git-wip-us.apache.org/repos/asf/camel/blob/0c126492/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/joda/DateTimeDeserializer.java
--
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/joda/DateTimeDeserializer.java
 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/joda/DateTimeDeserializer.java
new file mode 100644
index 000..280f238
--- /dev/null
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/joda/DateTimeDeserializer.java
@@ -0,0 +1,64 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+

[3/4] camel git commit: CAMEL-9638: ignored test for external SNIHostName site

2016-06-24 Thread anoordover
CAMEL-9638: ignored test for external SNIHostName site


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/8ba68e34
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/8ba68e34
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/8ba68e34

Branch: refs/heads/master
Commit: 8ba68e34a397c69e78f461b4fb274f457cd81e37
Parents: c9350c6
Author: Arno Noordover 
Authored: Fri Jun 24 19:15:50 2016 +0200
Committer: Arno Noordover 
Committed: Fri Jun 24 19:15:50 2016 +0200

--
 .../org/apache/camel/component/http4/HttpSNIHostNameTest.java | 3 +++
 1 file changed, 3 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/8ba68e34/components/camel-http4/src/test/java/org/apache/camel/component/http4/HttpSNIHostNameTest.java
--
diff --git 
a/components/camel-http4/src/test/java/org/apache/camel/component/http4/HttpSNIHostNameTest.java
 
b/components/camel-http4/src/test/java/org/apache/camel/component/http4/HttpSNIHostNameTest.java
index 7f9ffb4..65c4425 100644
--- 
a/components/camel-http4/src/test/java/org/apache/camel/component/http4/HttpSNIHostNameTest.java
+++ 
b/components/camel-http4/src/test/java/org/apache/camel/component/http4/HttpSNIHostNameTest.java
@@ -19,12 +19,14 @@ package org.apache.camel.component.http4;
 import org.apache.camel.CamelExecutionException;
 import org.apache.camel.http.common.HttpOperationFailedException;
 import org.apache.camel.test.spring.CamelSpringTestSupport;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.springframework.context.support.AbstractApplicationContext;
 import org.springframework.context.support.ClassPathXmlApplicationContext;
 
 import static org.hamcrest.core.Is.is;
 
+@Ignore("Ignored test because of external dependency.")
 public class HttpSNIHostNameTest extends CamelSpringTestSupport {
 
 @Test
@@ -32,6 +34,7 @@ public class HttpSNIHostNameTest extends 
CamelSpringTestSupport {
 String result = template.requestBody("direct:goodSNI", null, 
String.class);
 assertNotNull(result);
 }
+
 @Test
 public void testMnotDotNetNoSniDoesReturnStatusCode403() {
 try {



[2/4] camel git commit: CAMEL-9638: implemented review notes

2016-06-24 Thread anoordover
CAMEL-9638: implemented review notes


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/c9350c6f
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/c9350c6f
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/c9350c6f

Branch: refs/heads/master
Commit: c9350c6fdcb8bd7dacd014180a8cd801c01a9f21
Parents: 3d84bbe
Author: Arno Noordover 
Authored: Thu Jun 23 22:13:36 2016 +0200
Committer: Arno Noordover 
Committed: Thu Jun 23 22:13:36 2016 +0200

--
 .../camel/util/jsse/BaseSSLContextParameters.java   |  8 +---
 .../camel/util/jsse/SSLContextClientParameters.java |  6 +-
 .../AbstractSSLContextClientParametersFactoryBean.java  |  6 +++---
 .../core/xml/util/jsse/SNIHostNamesDefinition.java  |  4 ++--
 .../apache/camel/component/http4/CamelHttp4Context.xml  | 12 ++--
 5 files changed, 21 insertions(+), 15 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/c9350c6f/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
--
diff --git 
a/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
 
b/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
index a585fbe..cd1af45 100644
--- 
a/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
+++ 
b/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
@@ -522,9 +522,11 @@ public abstract class BaseSSLContextParameters extends 
JsseParameters {
 @Override
 public SSLSocket configure(SSLSocket socket) {
 
-SSLParameters sslParameters = socket.getSSLParameters();
-sslParameters.setServerNames(getSNIHostNames());
-socket.setSSLParameters(sslParameters);
+if (!getSNIHostNames().isEmpty()) {
+SSLParameters sslParameters = socket.getSSLParameters();
+sslParameters.setServerNames(getSNIHostNames());
+socket.setSSLParameters(sslParameters);
+}
 
 Collection filteredCipherSuites = 
BaseSSLContextParameters.this
 .filter(enabledCipherSuites, 
Arrays.asList(socket.getSSLParameters().getCipherSuites()),

http://git-wip-us.apache.org/repos/asf/camel/blob/c9350c6f/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
--
diff --git 
a/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
 
b/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
index a31fc21..0d48dbc 100644
--- 
a/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
+++ 
b/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
@@ -39,12 +39,16 @@ public class SSLContextClientParameters extends 
BaseSSLContextParameters {
 
 private List sniHostNames = new ArrayList<>();
 
-public void setSniHostNames(List sniHostNames) {
+public void addAllSniHostNames(List sniHostNames) {
 for (String sniHostName : sniHostNames) {
 this.sniHostNames.add(new SNIHostName(sniHostName));
 }
 }
 
+public void setSniHostName(String sniHostName) {
+this.sniHostNames.add(new SNIHostName(sniHostName));
+}
+
 @Override
 protected List getSNIHostNames() {
 return sniHostNames;

http://git-wip-us.apache.org/repos/asf/camel/blob/c9350c6f/components/camel-core-xml/src/main/java/org/apache/camel/core/xml/util/jsse/AbstractSSLContextClientParametersFactoryBean.java
--
diff --git 
a/components/camel-core-xml/src/main/java/org/apache/camel/core/xml/util/jsse/AbstractSSLContextClientParametersFactoryBean.java
 
b/components/camel-core-xml/src/main/java/org/apache/camel/core/xml/util/jsse/AbstractSSLContextClientParametersFactoryBean.java
index 55b465a..5778264 100644
--- 
a/components/camel-core-xml/src/main/java/org/apache/camel/core/xml/util/jsse/AbstractSSLContextClientParametersFactoryBean.java
+++ 
b/components/camel-core-xml/src/main/java/org/apache/camel/core/xml/util/jsse/AbstractSSLContextClientParametersFactoryBean.java
@@ -27,14 +27,14 @@ import 
org.apache.camel.util.jsse.SSLContextClientParameters;
 @XmlTransient
 public abstract class AbstractSSLContextClientParametersFactoryBean extends 
AbstractBaseSSLContextParametersFactoryBean {
 
-@XmlElement(name = "SNIHostNamesDefinition")
+@XmlElement(name = "sniHostNames")
 private SNIHostNamesDefinition sniHostNamesDefinition;
 
 @Override
 protected SSLContextClient

[4/4] camel git commit: This closes #1048 Merge branch 'CAMEL-9638'

2016-06-24 Thread anoordover
This closes #1048
Merge branch 'CAMEL-9638'


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/f39ba5fe
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/f39ba5fe
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/f39ba5fe

Branch: refs/heads/master
Commit: f39ba5feed14a10bad7b71047e8a5bed7e4e7653
Parents: 9c4e9df 8ba68e3
Author: Arno Noordover 
Authored: Fri Jun 24 21:23:16 2016 +0200
Committer: Arno Noordover 
Committed: Fri Jun 24 21:23:16 2016 +0200

--
 .../util/jsse/BaseSSLContextParameters.java | 18 -
 .../util/jsse/SSLContextClientParameters.java   | 20 +
 ...ctSSLContextClientParametersFactoryBean.java | 11 ++-
 .../xml/util/jsse/SNIHostNamesDefinition.java   | 36 +
 components/camel-http4/pom.xml  |  5 ++
 .../component/http4/HttpSNIHostNameTest.java| 64 
 .../camel/component/http4/CamelHttp4Context.xml | 77 
 7 files changed, 227 insertions(+), 4 deletions(-)
--




[1/4] camel git commit: CAMEL-9638: Introduced SNIHostName config in sslContextParameters

2016-06-24 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 9c4e9df98 -> f39ba5fee


CAMEL-9638: Introduced SNIHostName config in sslContextParameters


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/3d84bbef
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/3d84bbef
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/3d84bbef

Branch: refs/heads/master
Commit: 3d84bbef8879fc7a64c73ad38aae6c20099f1d73
Parents: b8f5da7
Author: Arno Noordover 
Authored: Wed Jun 22 21:07:20 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 22 21:07:20 2016 +0200

--
 .../util/jsse/BaseSSLContextParameters.java | 16 +++-
 .../util/jsse/SSLContextClientParameters.java   | 16 
 ...ctSSLContextClientParametersFactoryBean.java | 11 ++-
 .../xml/util/jsse/SNIHostNamesDefinition.java   | 36 +
 components/camel-http4/pom.xml  |  5 ++
 .../component/http4/HttpSNIHostNameTest.java| 61 
 .../camel/component/http4/CamelHttp4Context.xml | 77 
 7 files changed, 218 insertions(+), 4 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/3d84bbef/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
--
diff --git 
a/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
 
b/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
index 0554067..a585fbe 100644
--- 
a/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
+++ 
b/camel-core/src/main/java/org/apache/camel/util/jsse/BaseSSLContextParameters.java
@@ -34,9 +34,12 @@ import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
 import javax.net.ssl.KeyManager;
+import javax.net.ssl.SNIHostName;
+import javax.net.ssl.SNIServerName;
 import javax.net.ssl.SSLContext;
 import javax.net.ssl.SSLContextSpi;
 import javax.net.ssl.SSLEngine;
+import javax.net.ssl.SSLParameters;
 import javax.net.ssl.SSLServerSocket;
 import javax.net.ssl.SSLServerSocketFactory;
 import javax.net.ssl.SSLSessionContext;
@@ -109,7 +112,10 @@ public abstract class BaseSSLContextParameters extends 
JsseParameters {
  * The optional {@link SSLSessionContext} timeout time for {@link 
javax.net.ssl.SSLSession}s in seconds.
  */
 private String sessionTimeout;
-
+
+protected List getSNIHostNames() {
+return Collections.emptyList();
+}
 
 /**
  * Returns the optional explicitly configured cipher suites for this 
configuration.
@@ -515,7 +521,11 @@ public abstract class BaseSSLContextParameters extends 
JsseParameters {
 
 @Override
 public SSLSocket configure(SSLSocket socket) {
-
+
+SSLParameters sslParameters = socket.getSSLParameters();
+sslParameters.setServerNames(getSNIHostNames());
+socket.setSSLParameters(sslParameters);
+
 Collection filteredCipherSuites = 
BaseSSLContextParameters.this
 .filter(enabledCipherSuites, 
Arrays.asList(socket.getSSLParameters().getCipherSuites()),
 Arrays.asList(socket.getEnabledCipherSuites()),
@@ -539,7 +549,7 @@ public abstract class BaseSSLContextParameters extends 
JsseParameters {
 Arrays.asList(socket.getEnabledProtocols()),
 enabledSecureSocketProtocolsPatterns, 
defaultEnabledSecureSocketProtocolsPatterns,
 !allowPassthrough);
-
+
 if (LOG.isDebugEnabled()) {
 LOG.debug(SSL_SOCKET_PROTOCOL_LOG_MSG,
 new Object[] {socket,

http://git-wip-us.apache.org/repos/asf/camel/blob/3d84bbef/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
--
diff --git 
a/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
 
b/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
index b8cca2f..a31fc21 100644
--- 
a/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
+++ 
b/camel-core/src/main/java/org/apache/camel/util/jsse/SSLContextClientParameters.java
@@ -17,9 +17,12 @@
 package org.apache.camel.util.jsse;
 
 import java.security.GeneralSecurityException;
+import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 
+import javax.net.ssl.SNIHostName;
+import javax.net.ssl.SNIServerName;
 import javax.net.ssl.SSLContext;
 import javax.net.ssl.SSLEngine;
 import javax.net.ssl.SSLServerSocketFactory;
@@ -34,6 +37,19 @@ public class SSLContext

camel git commit: CAMEL-10024: Cherry picked from master

2016-06-24 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/camel-2.17.x 38c75c5cc -> 1d3e4a2d6


CAMEL-10024: Cherry picked from master


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/1d3e4a2d
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/1d3e4a2d
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/1d3e4a2d

Branch: refs/heads/camel-2.17.x
Commit: 1d3e4a2d6dff57b9487574972df8098cf0d2adcd
Parents: 38c75c5
Author: Arno Noordover 
Authored: Sat Jun 18 17:11:31 2016 +0200
Committer: Arno Noordover 
Committed: Fri Jun 24 21:06:44 2016 +0200

--
 .../camel/component/mina2/Mina2Consumer.java|  6 +-
 .../camel/component/mina2/Mina2Producer.java| 74 +++-
 .../component/mina2/Mina2TextLineDelimiter.java | 18 -
 ...Mina2ClientModeTcpTextlineDelimiterTest.java |  2 +-
 .../mina2/Mina2DisconnectRaceConditionTest.java | 70 ++
 .../component/mina2/Mina2EncodingTest.java  |  2 +-
 .../mina2/Mina2ExchangeTimeOutTest.java |  2 +-
 .../mina2/Mina2NoResponseFromServerTest.java|  4 +-
 .../mina2/Mina2ProducerShutdownMockTest.java|  7 +-
 .../mina2/Mina2ReverseProtocolHandler.java  |  2 +-
 .../mina2/Mina2TransferExchangeOptionTest.java  |  2 +-
 11 files changed, 141 insertions(+), 48 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/1d3e4a2d/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
--
diff --git 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
index 2e913e7..ac461da 100644
--- 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
+++ 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
@@ -115,7 +115,7 @@ public class Mina2Consumer extends DefaultConsumer {
 if (configuration.isClientMode() && 
configuration.getProtocol().equals("tcp")) {
 LOG.info("Disconnect from server address: {} using connector: {}", 
address, connector);
 if (session != null) {
-CloseFuture closeFuture = session.close(true);
+CloseFuture closeFuture = session.closeNow();
 closeFuture.awaitUninterruptibly();
 }
 connector.dispose(true);
@@ -382,7 +382,7 @@ public class Mina2Consumer extends DefaultConsumer {
 // close invalid session
 if (session != null) {
 LOG.warn("Closing session as an exception was thrown from 
MINA");
-session.close(true);
+session.closeNow();
 }
 
 // must wrap and rethrow since cause can be of Throwable and we 
must only throw Exception
@@ -456,7 +456,7 @@ public class Mina2Consumer extends DefaultConsumer {
 }
 if (disconnect) {
 LOG.debug("Closing session when complete at address: {}", 
address);
-session.close(true);
+session.closeNow();
 }
 }
 }

http://git-wip-us.apache.org/repos/asf/camel/blob/1d3e4a2d/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
--
diff --git 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
index 132abe3..8e3eaec 100644
--- 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
+++ 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
@@ -65,8 +65,10 @@ import org.slf4j.LoggerFactory;
 public class Mina2Producer extends DefaultProducer implements ServicePoolAware 
{
 
 private static final Logger LOG = 
LoggerFactory.getLogger(Mina2Producer.class);
+private final ResponseHandler handler;
 private IoSession session;
-private CountDownLatch latch;
+private CountDownLatch responseLatch;
+private CountDownLatch closeLatch;
 private boolean lazySessionCreation;
 private long timeout;
 private SocketAddress address;
@@ -93,6 +95,8 @@ public class Mina2Producer extends DefaultProducer implements 
ServicePoolAware {
 } else if (protocol.equals("vm")) {
 setupVmProtocol(protocol);
 }
+handler = new ResponseHandler();
+connector.setHandler(handler);
 }
 
 @Override
@@ -143,10 +147,9 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 
 // if sync is true then we sho

camel git commit: CS fix

2016-06-22 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 7fd93bb8c -> b8f5da747


CS fix


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/b8f5da74
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/b8f5da74
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/b8f5da74

Branch: refs/heads/master
Commit: b8f5da74760d60b007bef15ac8c52b824a925aa1
Parents: 7fd93bb
Author: Arno Noordover 
Authored: Wed Jun 22 17:08:55 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 22 17:08:55 2016 +0200

--
 .../commands/AbstractLocalCamelControllerTest.java  | 16 
 1 file changed, 16 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/b8f5da74/platforms/commands/commands-core/src/test/java/org/apache/camel/commands/AbstractLocalCamelControllerTest.java
--
diff --git 
a/platforms/commands/commands-core/src/test/java/org/apache/camel/commands/AbstractLocalCamelControllerTest.java
 
b/platforms/commands/commands-core/src/test/java/org/apache/camel/commands/AbstractLocalCamelControllerTest.java
index c55be8b..71886c1 100644
--- 
a/platforms/commands/commands-core/src/test/java/org/apache/camel/commands/AbstractLocalCamelControllerTest.java
+++ 
b/platforms/commands/commands-core/src/test/java/org/apache/camel/commands/AbstractLocalCamelControllerTest.java
@@ -1,3 +1,19 @@
+/**
+ * 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.commands;
 
 import java.util.List;



[2/3] camel git commit: CAMEL-10024: Renamed latch and null-safety

2016-06-22 Thread anoordover
CAMEL-10024: Renamed latch and null-safety


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/4d14c575
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/4d14c575
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/4d14c575

Branch: refs/heads/master
Commit: 4d14c575057f5f4610605f2cda0254c5e8c71885
Parents: fe41b1b
Author: Arno Noordover 
Authored: Sat Jun 18 17:11:31 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jun 18 17:11:31 2016 +0200

--
 .../camel/component/mina2/Mina2Producer.java| 42 
 .../component/mina2/Mina2TextLineDelimiter.java | 18 -
 2 files changed, 34 insertions(+), 26 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/4d14c575/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
--
diff --git 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
index 4337075..2ac6e92 100644
--- 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
+++ 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
@@ -67,7 +67,8 @@ public class Mina2Producer extends DefaultProducer implements 
ServicePoolAware {
 private static final Logger LOG = 
LoggerFactory.getLogger(Mina2Producer.class);
 private final ResponseHandler handler;
 private IoSession session;
-private CountDownLatch latch;
+private CountDownLatch responseLatch;
+private CountDownLatch closeLatch;
 private boolean lazySessionCreation;
 private long timeout;
 private SocketAddress address;
@@ -77,7 +78,6 @@ public class Mina2Producer extends DefaultProducer implements 
ServicePoolAware {
 private Mina2Configuration configuration;
 private IoSessionConfig connectorConfig;
 private ExecutorService workerPool;
-private CountDownLatch closeLatch;
 
 public Mina2Producer(Mina2Endpoint endpoint) throws Exception {
 super(endpoint);
@@ -147,8 +147,8 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 
 // if sync is true then we should also wait for a response 
(synchronous mode)
 if (sync) {
-// only initialize latch if we should get a response
-latch = new CountDownLatch(1);
+// only initialize responseLatch if we should get a response
+responseLatch = new CountDownLatch(1);
 // reset handler if we expect a response
 handler.reset();
 }
@@ -168,7 +168,7 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 if (sync) {
 // wait for response, consider timeout
 LOG.debug("Waiting for response using timeout {} millis.", 
timeout);
-boolean done = latch.await(timeout, TimeUnit.MILLISECONDS);
+boolean done = responseLatch.await(timeout, TimeUnit.MILLISECONDS);
 if (!done) {
 throw new ExchangeTimedOutException(exchange, timeout);
 }
@@ -421,21 +421,7 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 if (delimiter == null) {
 return LineDelimiter.DEFAULT;
 }
-
-switch (delimiter) {
-case DEFAULT:
-return LineDelimiter.DEFAULT;
-case AUTO:
-return LineDelimiter.AUTO;
-case UNIX:
-return LineDelimiter.UNIX;
-case WINDOWS:
-return LineDelimiter.WINDOWS;
-case MAC:
-return LineDelimiter.MAC;
-default:
-throw new IllegalArgumentException("Unknown textline delimiter: " 
+ delimiter);
-}
+return delimiter.getLineDelimiter();
 }
 
 private Charset getEncodingParameter(String type, Mina2Configuration 
configuration) {
@@ -492,11 +478,11 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 this.message = message;
 messageReceived = true;
 cause = null;
-countDown();
+notifyResultAvailable();
 }
 
-protected void countDown() {
-CountDownLatch downLatch = latch;
+protected void notifyResultAvailable() {
+CountDownLatch downLatch = responseLatch;
 if (downLatch != null) {
 downLatch.countDown();
 }
@@ -509,9 +495,15 @@ public class Mina2Producer extends DefaultProducer 
implements ServicePoolAware {
 LOG.debug("Session closed but no message received from 
address: {}", address);
  

[1/3] camel git commit: CAMEL-10024: sync on close and deprecation

2016-06-22 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 8f2bc1514 -> 76544116f


CAMEL-10024: sync on close and deprecation


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/fe41b1bb
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/fe41b1bb
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/fe41b1bb

Branch: refs/heads/master
Commit: fe41b1bb9ac743214a6f6baba393a251e55037fc
Parents: e784761
Author: Arno Noordover 
Authored: Fri Jun 17 16:47:14 2016 +0200
Committer: Arno Noordover 
Committed: Fri Jun 17 16:47:14 2016 +0200

--
 .../camel/component/mina2/Mina2Consumer.java|  6 +-
 .../camel/component/mina2/Mina2Producer.java| 36 +++---
 ...Mina2ClientModeTcpTextlineDelimiterTest.java |  2 +-
 .../mina2/Mina2DisconnectRaceConditionTest.java | 70 
 .../component/mina2/Mina2EncodingTest.java  |  2 +-
 .../mina2/Mina2ExchangeTimeOutTest.java |  2 +-
 .../mina2/Mina2NoResponseFromServerTest.java|  4 +-
 .../mina2/Mina2ProducerShutdownMockTest.java|  7 +-
 .../mina2/Mina2ReverseProtocolHandler.java  |  2 +-
 .../mina2/Mina2TransferExchangeOptionTest.java  |  2 +-
 10 files changed, 109 insertions(+), 24 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/fe41b1bb/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
--
diff --git 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
index c2a7c50..a916bab 100644
--- 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
+++ 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Consumer.java
@@ -115,7 +115,7 @@ public class Mina2Consumer extends DefaultConsumer {
 if (configuration.isClientMode() && 
configuration.getProtocol().equals("tcp")) {
 LOG.info("Disconnect from server address: {} using connector: {}", 
address, connector);
 if (session != null) {
-CloseFuture closeFuture = session.close(true);
+CloseFuture closeFuture = session.closeNow();
 closeFuture.awaitUninterruptibly();
 }
 connector.dispose(true);
@@ -382,7 +382,7 @@ public class Mina2Consumer extends DefaultConsumer {
 // close invalid session
 if (session != null) {
 LOG.warn("Closing session as an exception was thrown from 
MINA");
-session.close(true);
+session.closeNow();
 }
 
 // must wrap and rethrow since cause can be of Throwable and we 
must only throw Exception
@@ -456,7 +456,7 @@ public class Mina2Consumer extends DefaultConsumer {
 }
 if (disconnect) {
 LOG.debug("Closing session when complete at address: {}", 
address);
-session.close(true);
+session.closeNow();
 }
 }
 }

http://git-wip-us.apache.org/repos/asf/camel/blob/fe41b1bb/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
--
diff --git 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
index fdd02dc..4337075 100644
--- 
a/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
+++ 
b/components/camel-mina2/src/main/java/org/apache/camel/component/mina2/Mina2Producer.java
@@ -65,6 +65,7 @@ import org.slf4j.LoggerFactory;
 public class Mina2Producer extends DefaultProducer implements ServicePoolAware 
{
 
 private static final Logger LOG = 
LoggerFactory.getLogger(Mina2Producer.class);
+private final ResponseHandler handler;
 private IoSession session;
 private CountDownLatch latch;
 private boolean lazySessionCreation;
@@ -76,6 +77,7 @@ public class Mina2Producer extends DefaultProducer implements 
ServicePoolAware {
 private Mina2Configuration configuration;
 private IoSessionConfig connectorConfig;
 private ExecutorService workerPool;
+private CountDownLatch closeLatch;
 
 public Mina2Producer(Mina2Endpoint endpoint) throws Exception {
 super(endpoint);
@@ -93,6 +95,8 @@ public class Mina2Producer extends DefaultProducer implements 
ServicePoolAware {
 } else if (protocol.equals("vm")) {
 setupVmProtocol(protocol);
 }
+handler = new ResponseHandler();
+connector.setHandler(handler);
 }
 
 @Overr

[3/3] camel git commit: CAMEL-10024: sync on close and deprecation This closes #1043

2016-06-22 Thread anoordover
CAMEL-10024: sync on close and deprecation
This closes #1043


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/76544116
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/76544116
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/76544116

Branch: refs/heads/master
Commit: 76544116f587440bfa4df684242bbde895b99ce5
Parents: 8f2bc15 4d14c57
Author: Arno Noordover 
Authored: Wed Jun 22 14:33:38 2016 +0200
Committer: Arno Noordover 
Committed: Wed Jun 22 14:33:38 2016 +0200

--
 .../camel/component/mina2/Mina2Consumer.java|  6 +-
 .../camel/component/mina2/Mina2Producer.java| 74 +++-
 .../component/mina2/Mina2TextLineDelimiter.java | 18 -
 ...Mina2ClientModeTcpTextlineDelimiterTest.java |  2 +-
 .../mina2/Mina2DisconnectRaceConditionTest.java | 70 ++
 .../component/mina2/Mina2EncodingTest.java  |  2 +-
 .../mina2/Mina2ExchangeTimeOutTest.java |  2 +-
 .../mina2/Mina2NoResponseFromServerTest.java|  4 +-
 .../mina2/Mina2ProducerShutdownMockTest.java|  7 +-
 .../mina2/Mina2ReverseProtocolHandler.java  |  2 +-
 .../mina2/Mina2TransferExchangeOptionTest.java  |  2 +-
 11 files changed, 141 insertions(+), 48 deletions(-)
--




[1/2] camel git commit: CS fixes

2016-06-18 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 0e266627d -> 3e6e76c09


CS fixes


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/d046e935
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/d046e935
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/d046e935

Branch: refs/heads/master
Commit: d046e935ef98cea29269c736f3dac2478811c0a4
Parents: 26355f7
Author: Arno Noordover 
Authored: Sat Jun 18 21:30:13 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jun 18 21:30:13 2016 +0200

--
 .../java/org/apache/camel/component/mongodb/MongoDbProducer.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/d046e935/components/camel-mongodb/src/main/java/org/apache/camel/component/mongodb/MongoDbProducer.java
--
diff --git 
a/components/camel-mongodb/src/main/java/org/apache/camel/component/mongodb/MongoDbProducer.java
 
b/components/camel-mongodb/src/main/java/org/apache/camel/component/mongodb/MongoDbProducer.java
index f7663b8..76cfca1 100644
--- 
a/components/camel-mongodb/src/main/java/org/apache/camel/component/mongodb/MongoDbProducer.java
+++ 
b/components/camel-mongodb/src/main/java/org/apache/camel/component/mongodb/MongoDbProducer.java
@@ -271,7 +271,7 @@ public class MongoDbProducer extends DefaultProducer {
 }
 
 if (sortBy == null) {
-   sortBy = new BasicDBObject();
+sortBy = new BasicDBObject();
 }
 
 BasicDBObject ret = 
dbCol.find(o).projection(fieldFilter).sort(sortBy).first();



[2/2] camel git commit: Merge remote-tracking branch 'origin/master'

2016-06-18 Thread anoordover
Merge remote-tracking branch 'origin/master'


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/3e6e76c0
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/3e6e76c0
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/3e6e76c0

Branch: refs/heads/master
Commit: 3e6e76c09e4eda0767f8c3211ef4c5058ea38eca
Parents: d046e93 0e26662
Author: Arno Noordover 
Authored: Sat Jun 18 21:30:53 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jun 18 21:30:53 2016 +0200

--
 .../java/org/apache/camel/converter/jaxp/StaxConverterTest.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
--




camel git commit: CS fixes

2016-06-18 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master a7f07849d -> 26355f7ae


CS fixes


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/26355f7a
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/26355f7a
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/26355f7a

Branch: refs/heads/master
Commit: 26355f7ae105a9e27179f9e2a6c01894a8ea373a
Parents: a7f0784
Author: Arno Noordover 
Authored: Sat Jun 18 20:57:13 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jun 18 20:57:13 2016 +0200

--
 .../java/org/apache/camel/model/dataformat/JsonDataFormat.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/26355f7a/camel-core/src/main/java/org/apache/camel/model/dataformat/JsonDataFormat.java
--
diff --git 
a/camel-core/src/main/java/org/apache/camel/model/dataformat/JsonDataFormat.java
 
b/camel-core/src/main/java/org/apache/camel/model/dataformat/JsonDataFormat.java
index d28c423..6521c7c 100644
--- 
a/camel-core/src/main/java/org/apache/camel/model/dataformat/JsonDataFormat.java
+++ 
b/camel-core/src/main/java/org/apache/camel/model/dataformat/JsonDataFormat.java
@@ -315,10 +315,10 @@ public class JsonDataFormat extends DataFormatDefinition {
 setProperty(routeContext.getCamelContext(), this, 
"dataFormatName", "json-xstream");
 } else if (library == JsonLibrary.Jackson) {
 setProperty(routeContext.getCamelContext(), this, 
"dataFormatName", "json-jackson");
-} else if (library == JsonLibrary.Gson){
+} else if (library == JsonLibrary.Gson) {
 setProperty(routeContext.getCamelContext(), this, 
"dataFormatName", "json-gson");
 } else {
-   setProperty(routeContext.getCamelContext(), this, 
"dataFormatName", "json-johnzon");
+setProperty(routeContext.getCamelContext(), this, 
"dataFormatName", "json-johnzon");
 }
 
 if (unmarshalType == null && unmarshalTypeName != null) {



[2/2] camel git commit: This closes #1045

2016-06-18 Thread anoordover
This closes #1045


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/a7f07849
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/a7f07849
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/a7f07849

Branch: refs/heads/master
Commit: a7f07849d59c19e6379036fb57fa303396a9cd27
Parents: 70ea982 401a668
Author: Arno Noordover 
Authored: Sat Jun 18 20:26:57 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jun 18 20:26:57 2016 +0200

--
 .../src/main/java/org/apache/camel/component/sql/SqlConsumer.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
--




[1/2] camel git commit: fix misleading debug log message concerning the number of polled messages

2016-06-18 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master 70ea98262 -> a7f07849d


fix misleading debug log message concerning the number of polled messages


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/401a6686
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/401a6686
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/401a6686

Branch: refs/heads/master
Commit: 401a6686314248dc22257145e2feb24592670aa6
Parents: e784761
Author: Pascal Schumacher 
Authored: Fri Jun 17 18:30:18 2016 +0200
Committer: Pascal Schumacher 
Committed: Fri Jun 17 18:30:18 2016 +0200

--
 .../src/main/java/org/apache/camel/component/sql/SqlConsumer.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/401a6686/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlConsumer.java
--
diff --git 
a/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlConsumer.java
 
b/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlConsumer.java
index ab48133..bc72d6e 100644
--- 
a/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlConsumer.java
+++ 
b/components/camel-sql/src/main/java/org/apache/camel/component/sql/SqlConsumer.java
@@ -210,9 +210,8 @@ public class SqlConsumer extends 
ScheduledBatchPollingConsumer {
 public int processBatch(Queue exchanges) throws Exception {
 int total = exchanges.size();
 
-// limit if needed
 if (maxMessagesPerPoll > 0 && total == maxMessagesPerPoll) {
-log.debug("Limiting to maximum messages to poll " + 
maxMessagesPerPoll + " as there was more messages in this poll.");
+log.debug("Maximum messages to poll is {} and there were exactly 
{} messages in this poll.", maxMessagesPerPoll, total);
 }
 
 for (int index = 0; index < total && isBatchAllowed(); index++) {



[1/2] camel git commit: improve grammar of "Limiting maximum messages to poll..." debug log messages

2016-06-18 Thread anoordover
Repository: camel
Updated Branches:
  refs/heads/master d9088dd0e -> 4d005d9a5


improve grammar of "Limiting maximum messages to poll..." debug log messages


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/0151ad31
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/0151ad31
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/0151ad31

Branch: refs/heads/master
Commit: 0151ad31c561e0533831345f8dd13262bb9e6781
Parents: e784761
Author: Pascal Schumacher 
Authored: Fri Jun 17 17:39:16 2016 +0200
Committer: Pascal Schumacher 
Committed: Fri Jun 17 17:39:16 2016 +0200

--
 .../org/apache/camel/component/file/GenericFileConsumer.java   | 6 +++---
 .../java/org/apache/camel/component/hbase/HBaseConsumer.java   | 2 +-
 .../java/org/apache/camel/component/ibatis/IBatisConsumer.java | 2 +-
 .../main/java/org/apache/camel/component/jpa/JpaConsumer.java  | 2 +-
 .../java/org/apache/camel/component/krati/KratiConsumer.java   | 2 +-
 .../java/org/apache/camel/component/mail/MailConsumer.java | 2 +-
 .../org/apache/camel/component/mybatis/MyBatisConsumer.java| 2 +-
 .../main/java/org/apache/camel/component/sql/SqlConsumer.java  | 2 +-
 8 files changed, 10 insertions(+), 10 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/camel/blob/0151ad31/camel-core/src/main/java/org/apache/camel/component/file/GenericFileConsumer.java
--
diff --git 
a/camel-core/src/main/java/org/apache/camel/component/file/GenericFileConsumer.java
 
b/camel-core/src/main/java/org/apache/camel/component/file/GenericFileConsumer.java
index bf7dc23..72d4ed1 100644
--- 
a/camel-core/src/main/java/org/apache/camel/component/file/GenericFileConsumer.java
+++ 
b/camel-core/src/main/java/org/apache/camel/component/file/GenericFileConsumer.java
@@ -145,7 +145,7 @@ public abstract class GenericFileConsumer extends 
ScheduledBatchPollingConsum
 
 // log if we hit the limit
 if (limitHit) {
-log.debug("Limiting maximum messages to poll at {} files as there 
was more messages in this poll.", maxMessagesPerPoll);
+log.debug("Limiting maximum messages to poll at {} files as there 
were more messages in this poll.", maxMessagesPerPoll);
 }
 
 // sort files using file comparator if provided
@@ -176,7 +176,7 @@ public abstract class GenericFileConsumer extends 
ScheduledBatchPollingConsum
 // we are not eager limiting, but we have configured a limit, so cut 
the list of files
 if (!eagerLimitMaxMessagesPerPoll && maxMessagesPerPoll > 0) {
 if (files.size() > maxMessagesPerPoll) {
-log.debug("Limiting maximum messages to poll at {} files as 
there was more messages in this poll.", maxMessagesPerPoll);
+log.debug("Limiting maximum messages to poll at {} files as 
there were more messages in this poll.", maxMessagesPerPoll);
 // must first remove excessive files from the in progress 
repository
 removeExcessiveInProgressFiles(q, maxMessagesPerPoll);
 }
@@ -201,7 +201,7 @@ public abstract class GenericFileConsumer extends 
ScheduledBatchPollingConsum
 
 // limit if needed
 if (maxMessagesPerPoll > 0 && total > maxMessagesPerPoll) {
-log.debug("Limiting to maximum messages to poll {} as there was {} 
messages in this poll.", maxMessagesPerPoll, total);
+log.debug("Limiting to maximum messages to poll {} as there were 
{} messages in this poll.", maxMessagesPerPoll, total);
 total = maxMessagesPerPoll;
 }
 

http://git-wip-us.apache.org/repos/asf/camel/blob/0151ad31/components/camel-hbase/src/main/java/org/apache/camel/component/hbase/HBaseConsumer.java
--
diff --git 
a/components/camel-hbase/src/main/java/org/apache/camel/component/hbase/HBaseConsumer.java
 
b/components/camel-hbase/src/main/java/org/apache/camel/component/hbase/HBaseConsumer.java
index b54b284..2d5f8ba 100644
--- 
a/components/camel-hbase/src/main/java/org/apache/camel/component/hbase/HBaseConsumer.java
+++ 
b/components/camel-hbase/src/main/java/org/apache/camel/component/hbase/HBaseConsumer.java
@@ -153,7 +153,7 @@ public class HBaseConsumer extends 
ScheduledBatchPollingConsumer {
 
 // limit if needed
 if (maxMessagesPerPoll > 0 && total > maxMessagesPerPoll) {
-LOG.debug("Limiting to maximum messages to poll {} as there was {} 
messages in this poll.", maxMessagesPerPoll, total);
+LOG.debug("Limiting to maximum messages to poll {} as there were 
{} messages in this poll.", maxMessagesPerPoll, total);
 total = maxMessagesPerPoll;
 }
 

http://git-wip-us.apache.org/

[2/2] camel git commit: This closes #1044

2016-06-18 Thread anoordover
This closes #1044


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/4d005d9a
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/4d005d9a
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/4d005d9a

Branch: refs/heads/master
Commit: 4d005d9a5c7089d0b0ceabad53d7f77ecf7d3e4a
Parents: d9088dd 0151ad3
Author: Arno Noordover 
Authored: Sat Jun 18 20:09:10 2016 +0200
Committer: Arno Noordover 
Committed: Sat Jun 18 20:09:10 2016 +0200

--
 .../org/apache/camel/component/file/GenericFileConsumer.java   | 6 +++---
 .../java/org/apache/camel/component/hbase/HBaseConsumer.java   | 2 +-
 .../java/org/apache/camel/component/ibatis/IBatisConsumer.java | 2 +-
 .../main/java/org/apache/camel/component/jpa/JpaConsumer.java  | 2 +-
 .../java/org/apache/camel/component/krati/KratiConsumer.java   | 2 +-
 .../java/org/apache/camel/component/mail/MailConsumer.java | 2 +-
 .../org/apache/camel/component/mybatis/MyBatisConsumer.java| 2 +-
 .../main/java/org/apache/camel/component/sql/SqlConsumer.java  | 2 +-
 8 files changed, 10 insertions(+), 10 deletions(-)
--