Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix #3858 rename to quarkus.camel.source-location-enabled #3886

Merged
merged 1 commit into from
Jun 30, 2022
Merged

Fix #3858 rename to quarkus.camel.source-location-enabled #3886

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 6 additions & 0 deletions docs/modules/ROOT/pages/reference/extensions/core.adoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -254,6 +254,12 @@ Whether to enable the bridging of Camel events to CDI events.
| `boolean`
| `true`

|icon:lock[title=Fixed at build time] [[quarkus.camel.source-location-enabled]]`link:#quarkus.camel.source-location-enabled[quarkus.camel.source-location-enabled]`

Build time configuration options for enable/disable camel source location
| `boolean`
| `false`

|icon:lock[title=Fixed at build time] [[quarkus.camel.main.shutdown.timeout]]`link:#quarkus.camel.main.shutdown.timeout[quarkus.camel.main.shutdown.timeout]`

A timeout (with millisecond precision) to wait for `CamelMain++#++stop()` to finish
Expand Down
6 changes: 4 additions & 2 deletions ...loyment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelContextProcessor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ public void devModeCamelContextCustomizations(
}

/**
* Enable source location if camel.quarkus.source-location-enabled=true
* Enable source location if quarkus.camel.source-location-enabled=true
*
* @param recorder the recorder
* @param producer producer of context customizer build item
Expand Down Expand Up @@ -177,9 +177,11 @@ public boolean getAsBoolean() {
}

public static final class SourceLocationEnabled implements BooleanSupplier {
CamelConfig config;

@Override
public boolean getAsBoolean() {
return CamelSupport.getOptionalConfigValue("camel.quarkus.source-location-enabled", Boolean.class, false);
return config.sourceLocationEnabled;
}
}
}
2 changes: 1 addition & 1 deletion ...yment/src/test/java/org/apache/camel/quarkus/core/deployment/CamelSourceLocationTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public static Asset applicationProperties() {
Writer writer = new StringWriter();

Properties props = new Properties();
props.setProperty("camel.quarkus.source-location-enabled", "true");
props.setProperty("quarkus.camel.source-location-enabled", "true");

try {
props.store(writer, "");
Expand Down
6 changes: 6 additions & 0 deletions extensions-core/core/runtime/src/main/java/org/apache/camel/quarkus/core/CamelConfig.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,12 @@ public enum FailureRemedy {
@ConfigItem
public EventBridgeConfig eventBridge;

/**
* Build time configuration options for enable/disable camel source location
*/
@ConfigItem(defaultValue = "false")
public boolean sourceLocationEnabled;

@ConfigGroup
public static class BootstrapConfig {
/**
Expand Down