pub-solar/infra
6
1
Fork 3
Code Issues 16 Pull requests 6 Actions Packages Projects 1 Releases Wiki Activity

fix(nextcloud): push server is not a trusted proxy #62

Merged
teutat3s merged 1 commit from fix/nextcloud-trusted-proxies into main 2023-11-18 22:12:50 +00:00
Conversation 0 Commits 1 Files changed 1 +1 -5
1 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
hosts/nachtigall/apps/nextcloud.nix
View file

@ -36,6 +36,7 @@
notify_push = { notify_push = {
enable = true; enable = true;
bendDomainToLocalhost = true;
}; };
config = { config = {
@ -46,11 +47,6 @@
dbname = "nextcloud"; dbname = "nextcloud";
dbtableprefix = "oc_"; dbtableprefix = "oc_";
overwriteProtocol = "https"; overwriteProtocol = "https";
trustedProxies = [
"127.0.0.1"
"::1"
];
b12f commented 2023-11-18 21:34:21 +00:00
Review
Copy link

don't we still need this because we run nextcloud behind nginx?

don't we still need this because we run nextcloud behind nginx?
teutat3s commented 2023-11-18 21:44:14 +00:00
Review
Copy link

This option does it for us:
https://github.com/NixOS/nixpkgs/blob/nixos-23.05/nixos/modules/services/web-apps/nextcloud-notify_push.nix#L118-L120

This option does it for us: https://github.com/NixOS/nixpkgs/blob/nixos-23.05/nixos/modules/services/web-apps/nextcloud-notify_push.nix#L118-L120
}; };
extraOptions = { extraOptions = {