Skip to content

Add REDIS_HOST_USER variable to specify a redis user#2359

Merged
J0WI merged 1 commit intonextcloud:masterfrom
derhagen:master
Jan 11, 2025
Merged

Add REDIS_HOST_USER variable to specify a redis user#2359
J0WI merged 1 commit intonextcloud:masterfrom
derhagen:master

Conversation

@derhagen
Copy link
Contributor

@derhagen derhagen commented Jan 9, 2025

Our setup requires a 'user' => 'TheRealRedisUser' entry in config.php. This environment variable would make that easier.

@derhagen derhagen marked this pull request as ready for review January 10, 2025 14:08
@derhagen
Copy link
Contributor Author

@J0WI the errors from the previous run should be fixed now. feel free to start the test runner again.

btw is there any easy way to run your tests locally?

@J0WI
Copy link
Contributor

J0WI commented Jan 10, 2025

btw is there any easy way to run your tests locally?

sure: https://github.com/docker-library/official-images/tree/master/test

@derhagen
Copy link
Contributor Author

29.0.10-fpm-alpine looks like a flaky test to me. I don't quite get what went wrong in 29.0.10-fpm

@J0WI J0WI merged commit 5b932e3 into nextcloud:master Jan 11, 2025
18 of 21 checks passed
cuppett added a commit to cuppett/nextcloud-openshift-s2i that referenced this pull request Feb 16, 2026
Adds support for REDIS_HOST_USER environment variable to enable Redis ACL
user authentication. This is required for Redis 6+ when using ACL-based
authentication instead of the legacy password-only authentication.

Co-authored-by: Hagen <[email protected]>
Co-Authored-By: Claude Sonnet 4.5 (1M context) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants