matrix-org / synapse

Synapse: Matrix homeserver written in Python/Twisted.
https://matrix-org.github.io/synapse
Apache License 2.0
11.83k stars 2.13k forks source link

Add ENV condition for registration without verification #16414

Closed ParthJadhav closed 1 year ago

ParthJadhav commented 1 year ago

Pull Request Checklist

clokep commented 1 year ago

I think we're inclined not to accept this as we really don't want to make open registration easy to enable. This tends to lead to spam across the network and should generally not be enabled.

ParthJadhav commented 1 year ago

Hey @clokep , Thanks for the feedback. Although I was adding this to make it easier for Synapse to be hosted as an App on Umbrel App store.

https://github.com/getumbrel/umbrel-apps#2-%EF%B8%8Fpackaging-the-app-for-umbrel

It's already there but it's a really old version.

clokep commented 1 year ago

Although I was adding this to make it easier for Synapse to be hosted as an App on Umbrel App store.

I'm not sure how this makes it easier for hosting there; running with open registration is not a recommended configuration.