Repository: camel Updated Branches: refs/heads/camel-2.18.x 5a4f641b6 -> bb461ff34
CAMEL-10701 - Use proper classloader for loading authentication settings in Kafka Project: http://git-wip-us.apache.org/repos/asf/camel/repo Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/bb461ff3 Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/bb461ff3 Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/bb461ff3 Branch: refs/heads/camel-2.18.x Commit: bb461ff34c45e0e61f21790f738d29b582adb8c9 Parents: 5a4f641 Author: Antoine DESSAIGNE <antoine.dessai...@gmail.com> Authored: Fri Jan 13 10:33:25 2017 +0100 Committer: Andrea Cosentino <anco...@gmail.com> Committed: Fri Jan 13 11:04:44 2017 +0100 ---------------------------------------------------------------------- .../java/org/apache/camel/component/kafka/KafkaConsumer.java | 4 ++-- .../java/org/apache/camel/component/kafka/KafkaProducer.java | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/camel/blob/bb461ff3/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConsumer.java ---------------------------------------------------------------------- diff --git a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConsumer.java b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConsumer.java index 1614f5e..b94b156 100644 --- a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConsumer.java +++ b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConsumer.java @@ -117,8 +117,8 @@ public class KafkaConsumer extends DefaultConsumer { ClassLoader threadClassLoader = Thread.currentThread().getContextClassLoader(); try { - //Fix for running camel-kafka in OSGI see KAFKA-3218 - Thread.currentThread().setContextClassLoader(null); + // Kafka uses reflection for loading authentication settings, use its classloader + Thread.currentThread().setContextClassLoader(org.apache.kafka.clients.consumer.KafkaConsumer.class.getClassLoader()); this.consumer = new org.apache.kafka.clients.consumer.KafkaConsumer(kafkaProps); } finally { Thread.currentThread().setContextClassLoader(threadClassLoader); http://git-wip-us.apache.org/repos/asf/camel/blob/bb461ff3/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaProducer.java ---------------------------------------------------------------------- diff --git a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaProducer.java b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaProducer.java index 1a3ae93..2ffe96b 100644 --- a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaProducer.java +++ b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaProducer.java @@ -83,8 +83,8 @@ public class KafkaProducer extends DefaultAsyncProducer { if (kafkaProducer == null) { ClassLoader threadClassLoader = Thread.currentThread().getContextClassLoader(); try { - //Fix for running camel-kafka in OSGI see KAFKA-3218 - Thread.currentThread().setContextClassLoader(null); + // Kafka uses reflection for loading authentication settings, use its classloader + Thread.currentThread().setContextClassLoader(org.apache.kafka.clients.producer.KafkaProducer.class.getClassLoader()); kafkaProducer = new org.apache.kafka.clients.producer.KafkaProducer(props); } finally { Thread.currentThread().setContextClassLoader(threadClassLoader);
