spantaleev / matrix-docker-ansible-deploy

🐳 Matrix (An open network for secure, decentralized communication) server setup using Ansible and Docker
GNU Affero General Public License v3.0
4.9k stars 1.04k forks source link

Collapse matrix_dendrite_registration_disabled and matrix_synapse_enable_registration in one var #1615

Open TheLillo opened 2 years ago

TheLillo commented 2 years ago

Should be use something like matrix_homeserver_registration_disabled (or something like that) instead of having two different vars for the same feature #1620

aaronraimist commented 2 years ago

I understand that currently you can only choose one or the other but personally I would love the option to run two (or more!) homeserver implementations at the same time.

spantaleev commented 2 years ago

How would running 2 or more at the same time work?

aaronraimist commented 2 years ago

They would have to use separate domains of course but I wouldn't think it would be that difficult. Not sure how many people would use it though so may not be worth it.

It would be nice for me though to be able to have Synapse, Dendrite, and Conduit for testing things like https://github.com/vector-im/element-ios/pull/5510.

TheLillo commented 2 years ago

Well this request is for the policy, not for run more than one implementation at the same time. By default the two server's playbook (Dendrite and Synapse) have the opposite behavior. This could be an security issue for someone who migrate from one to the other. Having the same keyword would be great in term of policy definition