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.

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

Make Jasypt beans conditional so that can be overridden #1140

Merged
merged 2 commits into from
May 6, 2024

Conversation

Croway
Copy link
Contributor

@Croway Croway commented May 6, 2024

At the moment, the documentation does not work as is, a bean like

<bean id="jasypt" class="org.apache.camel.component.jasypt.JasyptPropertiesParser">
    <!-- password is mandatory, you can prefix it with sysenv: or sys: to indicate it should use
         an OS environment or JVM system property value, so you don't have the master password defined here -->
    <property name="password" value="secret"/>
</bean>

has to be defined as primary="true" otherwise two spring beans are loaded in the context for the same class.

Marking beans as conditional solves this issue.

@Croway Croway merged commit 3432a2a into apache:main May 6, 2024
1 of 2 checks passed
@davsclaus
Copy link
Contributor

Can you create a JIRA so we can track this and backport to LTS branches

@Croway
Copy link
Contributor Author

Croway commented May 6, 2024

Can you create a JIRA so we can track this and backport to LTS branches

sure, on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants