YunoHost-Apps / synapse_ynh

Matrix server (synapse) package for YunoHost
https://matrix.org/
GNU General Public License v3.0
79 stars 43 forks source link

Change the ssowat to fix a bug that prevents people from resetting their passwords #406

Closed tio-trom closed 3 months ago

tio-trom commented 9 months ago

In the file /etc/ssowat/conf.json we have

"synapse.admin_api": {
            "auth_header": false,
            "label": "matrix.trom.tf (Server administration API.)",
            "public": true,
            "show_tile": false,
            "uris": [
                "matrix.trom.tf/_matrix/_synapse"
            ],

But the URIS should be without _matrix, like "matrix.trom.tf/_synapse". If we do not fix this then no one can reset their passwords and the Synapse Admin does not work.

I have to do this change manually every-time Synapse or Yunohost get an update.

Can we please make sure it is fixed upstream?

Thanks!

Josue-T commented 9 months ago

Hello, normally it's already fixed by #301

tio-trom commented 9 months ago

Hi. Interestingly after every update the "bad" setting is back. I always have to fix it. Can you point where in the commit or the source files is the "uris" properly defined?

tio-trom commented 8 months ago

This is something....I have to fix this every single time. Anyone else has this issue? It drives me insane :D - because if I do not fix this no one can reset their passwords.... Any help please? The file /etc/ssowat/conf.json gets updated with the wrong info all of the time...

Thatoo commented 8 months ago

I don't face this issue. I don't have /_matrix/ before _synapse. I can't reproduce this issue. Are you sure this issue comes from this synapse package and not from Synapse Admin package for example?

tio-trom commented 8 months ago

Hm yeah I do not know....but one of them needs fixing :D . I will check tonight more in depth where it could come from.

tio-trom commented 8 months ago

So updates are updating that file....idk from where tho....

Josue-T commented 3 months ago

Fixed by #426