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

[ACS-2564] update docker-compose and community-docker-compose to use solr with secureComms=secret #655

Merged
merged 3 commits into from
Feb 22, 2022

Conversation

aonych
Copy link
Contributor

@aonych aonych commented Feb 18, 2022

Changed default solr SECURE COMMS from NONE to SECRET. We need to set the shared secret value for both alfresco -Dsolr.sharedSecret and solr -Dalfresco.secureComms.secret (default value is “secret”).

Copy link
Contributor

@alxgomz alxgomz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aonych aonych changed the title [ACS-2564] update docker-compose and community-docker-compose to use solo with secureComms=secret [ACS-2564] update docker-compose and community-docker-compose to use solr with secureComms=secret Feb 21, 2022
@aonych aonych merged commit dda1e3a into master Feb 22, 2022
@aonych aonych deleted the feature/ACS-2564_set_solr_secure_comms_to_secret branch February 22, 2022 09:02
p4km9y pushed a commit to xit-camp/acs-deployment that referenced this pull request Mar 25, 2022
…solr with secureComms=secret (Alfresco#655)

* [ACS-2564] update docker-compose and community-docker-compose to use solr with secureComms=secret

* [ACS-2564] fix alfresco-content-repository version

* [ACS-2564] change SOLR_OPTS to JAVA_TOOL_OPTIONS
alxgomz pushed a commit that referenced this pull request Jan 23, 2024
…solr with secureComms=secret (#655)

* [ACS-2564] update docker-compose and community-docker-compose to use solr with secureComms=secret

* [ACS-2564] fix alfresco-content-repository version

* [ACS-2564] change SOLR_OPTS to JAVA_TOOL_OPTIONS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants