New comment by TinfoilSubmarine on void-packages repository

https://github.com/void-linux/void-packages/pull/34676#issuecomment-1003061398

Comment:
> explicit is better than implicit because if you want to check logs you shouldn't need to know what's the default behaviour of vlogger

We'll need to have a Void team member weigh in on this because the history of the logging service (introduced in https://github.com/void-linux/void-packages/pull/31263) was what made me start using vlogger instead of the older `exec logger ...` pattern.

> If you read the [first three paragraphs of the config file that you want to install](https://github.com/matrix-org/synapse/blob/develop/docs/sample_config.yaml) you see that this config file is not intended to be copied into `/etc/synapse`, patched or otherwise. The config does contain several secrets that should be randomly generated and for this the correct and documented way is to use `generate_config`

I compared the output of `generate_config` with the patched `sample_config.yaml` and the only difference was that the header was included and the `log_config` location differed slightly. You are correct that this is confusing, so likely these paragraphs should be patched out as well. As far as secret generation, that is optional for having a working server, I specifically tested this. The comments above the three secrets that would be generated (`registration_shared_secret`, `macaroon_secret_key`, and `form_secret`) indicate that these are either optional or only needed for advanced functionality such as consent forms.