search

typokign / matrix-chart

Helm chart for deploying a Matrix homeserver stack
MIT License
88 stars 47 forks source link

Fix riot config.json mount path and server name configuration #25

Closed jjlpinto closed 4 years ago

jjlpinto commented 4 years ago

Based upon the riot container documentation: https://github.com/vector-im/riot-web#running-from-docker. The mount point for config.json should be /app/config.json and not /etc/riot-web/config.json.

I also changed the default_server_name to use the hostname from the values.yaml file instead of the serverName so that riot will point to the server URL even if it is different from the serverName variable.

typokign commented 4 years ago

Good catch on the mount point, that must have changed when the new Docker image was released.

RE the default homeserver, I did some research as I thought the default_server_name was for the location of .well-known/matrix/client. Turns out I was right, but more importantly, that field is deprecated! So I'll merge this for the config location change, but I'll update the config to use default_server_config before cutting a new release.

Thanks!