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

Fix condition for transferring key/trust store #163

Merged

Conversation

guidograzioli
Copy link
Member

@guidograzioli guidograzioli commented Sep 4, 2024

Fix the condition for transferring TLS stores, which would end in an error for paths defined as empty string.

This change also inverts nesting of single and double quotes in the sysconfig file for JAVA_ARGS: this allows to embed commands in it, like as follows (requires: activemq_systemd_expand_environment: true for the expansion to work):

-XX:HeapDumpPath=/var/log/amq-broker/amq-broker_$(date "+%Y%m%d_%H%M%S").hprof 

@guidograzioli guidograzioli added the bugfixes Fixes that resolve issues. SHOULD not be used for minor enhancements label Sep 4, 2024
@guidograzioli guidograzioli merged commit 0995705 into ansible-middleware:main Sep 4, 2024
20 checks passed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfixes Fixes that resolve issues. SHOULD not be used for minor enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant