Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Conversation

xddq
Copy link

@xddq xddq commented May 24, 2021

Fixes small config error.

Currently if we are creating a fresh configuration for matrix (based on the :latest docker image and the suggested commands) and then run the image with the suggested commands, we get a permission error.

PermissionError: [Errno 13] Permission denied: '/homeserver.log'.

issues:

10023
9970

What changed?

  • added the LOG_FILE_PATH env to the readme.md for the docker installation (documentation).
  • changed the default path from /homeserver.log to /data/homeserver.log (configuration)
  • I added a folder for you to easier test this issue. Be aware that running the script will delete the default volume that is described in the docker readme ("synapse-data").. I will remove this folder afterwards, if you think the change is okay.

@richvdh
Copy link
Member

richvdh commented May 24, 2021

nb this duplicates #10045.

@richvdh
Copy link
Member

richvdh commented May 24, 2021

thank you for the contribution, but #10045 has got there first :)

@richvdh richvdh closed this May 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default generated logger config causes "Permission denied: '/homeserver.log'" for non-root containers

2 participants