You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2025/01/13 20:54:50 UTC
(camel) branch main updated: CAMEL-21616: camel-main - Route exclude pattern does not work for discovered java classes
This is an automated email from the ASF dual-hosted git repository.
davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/main by this push:
new da98b324062 CAMEL-21616: camel-main - Route exclude pattern does not work for discovered java classes
da98b324062 is described below
commit da98b324062fbfb27381c6325a149dd85e7f9abf
Author: Claus Ibsen <cl...@gmail.com>
AuthorDate: Mon Jan 13 21:53:40 2025 +0100
CAMEL-21616: camel-main - Route exclude pattern does not work for discovered java classes
---
.../org/apache/camel/main/RoutesConfigurer.java | 51 +++++++++++++++-------
1 file changed, 36 insertions(+), 15 deletions(-)
diff --git a/core/camel-main/src/main/java/org/apache/camel/main/RoutesConfigurer.java b/core/camel-main/src/main/java/org/apache/camel/main/RoutesConfigurer.java
index 68022f19af8..f7cbd2cd21a 100644
--- a/core/camel-main/src/main/java/org/apache/camel/main/RoutesConfigurer.java
+++ b/core/camel-main/src/main/java/org/apache/camel/main/RoutesConfigurer.java
@@ -38,7 +38,9 @@ import org.apache.camel.spi.RoutesLoader;
import org.apache.camel.spi.StartupStepRecorder;
import org.apache.camel.support.OrderedComparator;
import org.apache.camel.support.PluginHelper;
+import org.apache.camel.util.AntPathMatcher;
import org.apache.camel.util.FileUtil;
+import org.apache.camel.util.ObjectHelper;
import org.apache.camel.util.StopWatch;
import org.apache.camel.util.TimeUtils;
import org.slf4j.Logger;
@@ -193,22 +195,41 @@ public class RoutesConfigurer {
if (getBasePackageScan() != null) {
step = recorder.beginStep(RoutesConfigurer.class, "packageScan", "Routes Configurer");
- String[] pkgs = getBasePackageScan().split(",");
- Set<Class<?>> set = PluginHelper.getPackageScanClassResolver(camelContext)
- .findImplementations(RoutesBuilder.class, pkgs);
- for (Class<?> routeClazz : set) {
- try {
- Object builder = camelContext.getInjector().newInstance(routeClazz);
- if (builder instanceof RoutesBuilder routesBuilder) {
- routes.add(routesBuilder);
- } else {
- LOG.warn("Class {} is not a RouteBuilder class", routeClazz);
+
+ boolean scan = true;
+ final String[] includes = javaRoutesIncludePattern != null ? javaRoutesIncludePattern.split(",") : null;
+ final String[] excludes = javaRoutesExcludePattern != null ? javaRoutesExcludePattern.split(",") : null;
+ if (includes != null && ObjectHelper.equal("false", javaRoutesIncludePattern)) {
+ scan = false;
+ }
+ if (scan) {
+ String[] pkgs = getBasePackageScan().split(",");
+ Set<Class<?>> set = PluginHelper.getPackageScanClassResolver(camelContext)
+ .findImplementations(RoutesBuilder.class, pkgs);
+ for (Class<?> routeClazz : set) {
+ // exclude take precedence over includes
+ String path = routeClazz.getName().replace(".", "/");
+ if (excludes != null && !"false".equals(javaRoutesExcludePattern)
+ && AntPathMatcher.INSTANCE.anyMatch(excludes, path)) {
+ continue;
}
- } catch (Exception e) {
- if (isIgnoreLoadingError()) {
- LOG.warn("Ignore loading error due to: {}. This exception is ignored.", e.getMessage());
- } else {
- throw RuntimeCamelException.wrapRuntimeException(e);
+ if (includes != null && !"false".equals(javaRoutesIncludePattern)
+ && !AntPathMatcher.INSTANCE.anyMatch(includes, path)) {
+ continue;
+ }
+ try {
+ Object builder = camelContext.getInjector().newInstance(routeClazz);
+ if (builder instanceof RoutesBuilder routesBuilder) {
+ routes.add(routesBuilder);
+ } else {
+ LOG.warn("Class {} is not a RouteBuilder class", routeClazz);
+ }
+ } catch (Exception e) {
+ if (isIgnoreLoadingError()) {
+ LOG.warn("Ignore loading error due to: {}. This exception is ignored.", e.getMessage());
+ } else {
+ throw RuntimeCamelException.wrapRuntimeException(e);
+ }
}
}
}