fix: handle redis authentication for healthcheck command #10907
Merged
+9
−4
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #10898
This PR includes the following changes:
REDISCLI_AUTH
parameterenv.example
with the overall formatdocker-compose
command parameters to ensure proper variable substitution, according to:Brief explanation: The
env_file
parameter is used to pass environment variables to containers, while variables in the docker compose yaml file like${REDIS_PASSWORD:-difyai123456}
require environment variables duringdocker-compose
command execution. Since the middleware env file in the documentation won't be automatically loaded as a.env
file, the--env-file
flag is necessary for docker-compose commands to properly substitute default values with the complete environment variables.Tip
Close issue syntax:
Fixes #<issue number>
orResolves #<issue number>
, see documentation for more details.Screenshots
Checklist
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint gods