Skip to content

Use default redis socket name in example config#10742

Merged
susnux merged 2 commits into
nextcloud:masterfrom
emcauliffe:patch-1
Nov 13, 2024
Merged

Use default redis socket name in example config#10742
susnux merged 2 commits into
nextcloud:masterfrom
emcauliffe:patch-1

Conversation

@emcauliffe

Copy link
Copy Markdown
Contributor

☑️ Resolves

🖼️ Screenshots

Before
Before

After
After

@kesselb

kesselb commented Aug 3, 2023

Copy link
Copy Markdown
Contributor

Thanks for your pull request 👍

Update example config to connect to default redis-server.sock file

The socket path/name is often adjusted by the package maintainer to match the system's default.
Default value provided from redis.conf (https://raw.githubusercontent.com/redis/redis/7.0/redis.conf) is /run/redis.sock

Matches https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/caching_configuration.html#connecting-to-single-redis-server-over-unix-socket

Not anymore? ;) The linked article says '/run/redis/redis-server.sock', 🤔

Could you please update your pull request to align the socket paths?

Comment thread admin_manual/configuration_files/files_locking_transactional.rst Outdated
emcauliffe and others added 2 commits November 13, 2024 02:00
Signed-off-by: Ethan McAuliffe <14607273+emcauliffe@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux merged commit 22aa3c1 into nextcloud:master Nov 13, 2024
@welcome

welcome Bot commented Nov 13, 2024

Copy link
Copy Markdown

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/documentation/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

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.

8 participants