From fe9fb739a9fef3ffe5de6a935bcd19e710351c89 Mon Sep 17 00:00:00 2001
From: Felix Buehler <account@buehler.rocks>
Date: Wed, 19 Oct 2022 23:34:26 +0200
Subject: [PATCH] freshrss: fix greader-api
---
nixos/modules/services/web-apps/freshrss.nix | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/nixos/modules/services/web-apps/freshrss.nix b/nixos/modules/services/web-apps/freshrss.nix
index 68780a3fe9dd..fe3b464ffc38 100644
--- a/nixos/modules/services/web-apps/freshrss.nix
+++ b/nixos/modules/services/web-apps/freshrss.nix
@@ -155,9 +155,17 @@ in
virtualHosts.${cfg.virtualHost} = {
root = "${cfg.package}/p";
+ # php files handling
+ # this regex is mandatory because of the API
locations."~ ^.+?\.php(/.*)?$".extraConfig = ''
fastcgi_pass unix:${config.services.phpfpm.pools.${cfg.pool}.socket};
fastcgi_split_path_info ^(.+\.php)(/.*)$;
+ # By default, the variable PATH_INFO is not set under PHP-FPM
+ # But FreshRSS API greader.php need it. If you have a “Bad Request” error, double check this var!
+ # NOTE: the separate $path_info variable is required. For more details, see:
+ # https://trac.nginx.org/nginx/ticket/321
+ set $path_info $fastcgi_path_info;
+ fastcgi_param PATH_INFO $path_info;
include ${pkgs.nginx}/conf/fastcgi_params;
include ${pkgs.nginx}/conf/fastcgi.conf;
'';