Hard to follow upstream homserver.yaml? #33

Closed
opened 2022-07-15 14:08:09 +00:00 by decentral1se · 3 comments
Owner

Related to #32, I've seen that in https://github.com/matrix-org/synapse/pull/12941 they've removed all the new options from the sample_config.yml in new releases.

This means, it is not possible to check all the homeserver.yaml configuration changes and therefore adapt to new changes via git log .... Especially for security changes.

It seems that we now have to follow the change log and then look into each PR, to see what is the name / default of the new option? Does anyone have any tips here? I'm not quite sure how to keep up now.

I've just released 1.62.0 and it deployed find. But maybe some default config is on that shouldn't be? Feels a bit unstable now tbh.

/cc @yksflip @kawaiipunk

Related to https://git.coopcloud.tech/coop-cloud/matrix-synapse/issues/32, I've seen that in https://github.com/matrix-org/synapse/pull/12941 they've removed all the new options from the `sample_config.yml` in new releases. This means, it is not possible to check all the `homeserver.yaml` configuration changes and therefore adapt to new changes via `git log ...`. Especially for security changes. It seems that we now have to follow the change log and then look into each PR, to see what is the name / default of the new option? Does anyone have any tips here? I'm not quite sure how to keep up now. I've just released 1.62.0 and it deployed find. But maybe some default config is on that shouldn't be? Feels a bit unstable now tbh. /cc @yksflip @kawaiipunk
Member

Perhaps the package maintenance commit log of Cloudron's Synapse can be of some use? https://git.cloudron.io/cloudron/synapse-app/-/commits/master

Perhaps the package maintenance commit log of Cloudron's Synapse can be of some use? https://git.cloudron.io/cloudron/synapse-app/-/commits/master
Author
Owner

Nice @josef! I see they are doing a stripped config (no comments, no random defaults) in https://git.cloudron.io/cloudron/synapse-app/-/blob/master/homeserver.yaml.template which we could also consider, I guess. But the concerns in https://github.com/matrix-org/synapse/pull/12941 still apply, some defaults might flip from true/false and we wouldn't really know unless we're following the change log + commit log of upstream closely... there might be no way around that for now until upstream change the way theyre doing things?

Nice @josef! I see they are doing a stripped config (no comments, no random defaults) in https://git.cloudron.io/cloudron/synapse-app/-/blob/master/homeserver.yaml.template which we could also consider, I guess. But the concerns in https://github.com/matrix-org/synapse/pull/12941 still apply, some defaults might flip from true/false and we wouldn't really know unless we're following the change log + commit log of upstream closely... there might be no way around that for now until upstream change the way theyre doing things?
Author
Owner

Ping: I am gonna make a move on this soon-ish. If upstream is reducing the config size, I'm inclined to follow them. Then we use our collective brain on the recipe side to make good decisions for privacy and such as we go forward with it.

Ping: I am gonna make a move on this soon-ish. If upstream is reducing the config size, I'm inclined to follow them. Then we use our collective brain on the recipe side to make good decisions for privacy and such as we go forward with it.
Sign in to join this conversation.
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/matrix-synapse#33
No description provided.