fix/add-matrix-wellknown #52

Merged
b12f merged 2 commits from fix/add-matrix-wellknown into main 2023-11-08 15:33:22 +00:00
Showing only changes of commit 6fa03994b1 - Show all commits

View file

@ -6,8 +6,8 @@ let
add_header X-XSS-Protection "1; mode=block"; add_header X-XSS-Protection "1; mode=block";
''; '';
clientConfig = import ./matrix/element-client-config.nix; clientConfig = import ./matrix/element-client-config.nix;
wellKnownClient."m.homeserver".base_url = "https://matrix.test.pub.solar"; wellKnownClient."m.homeserver".base_url = "https://matrix.pub.solar";
wellKnownServer."m.server" = "matrix.test.pub.solar:8448"; wellKnownServer."m.server" = "matrix.pub.solar:8448";
mkWellKnown = data: '' mkWellKnown = data: ''
teutat3s marked this conversation as resolved
Review

This breaks federation for the testing domain that @axeman is working on. Can we have an extra nginx vhost for testing and the current matrix homeserver?

This breaks federation for the testing domain that @axeman is working on. Can we have an extra nginx vhost for testing and the current matrix homeserver?
Review

I don't see this currently actively being worked on, so I'd rather that gets added/changed in a separate Pr so we get this merged

I don't see this currently actively being worked on, so I'd rather that gets added/changed in a separate Pr so we get this merged
Review

Okay then @axeman will have to add this back 👍

Okay then @axeman will have to add this back 👍
add_header Content-Type application/json; add_header Content-Type application/json;
add_header Access-Control-Allow-Origin *; add_header Access-Control-Allow-Origin *;
@ -20,12 +20,7 @@ let
in in
{ {
services.nginx.virtualHosts = { services.nginx.virtualHosts = {
"test.pub.solar" = { "pub.solar" = {
root = "/dev/null";
forceSSL = lib.mkDefault true;
enableACME = lib.mkDefault true;
locations = wellKnownLocations; locations = wellKnownLocations;
}; };