Skip to content

Commit

Permalink
nixos/sympa: fix PATH_INFO splitting for sympa-6.2.56
Browse files Browse the repository at this point in the history
Our regex for splitting HTTP path into SCRIPT_NAME and PATH_INFO was
incorrect when webLocation was set to "/". Since Sympa 6.2.56 this has
caused the web interface to return "421 Misdirected Request".

Since 6.2.56 Sympa can do the splitting on its own so we can simply
remove it from nginx configuration.

See also:
- sympa-community/sympa#879
- sympa-community/sympa#910
- sympa-community/sympa-community.github.io#53

(cherry picked from commit bd4e4dd)
  • Loading branch information
mmilata committed Jun 10, 2020
1 parent 4a644cc commit 3f31c0d
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions nixos/modules/services/mail/sympa.nix
Original file line number Diff line number Diff line change
Expand Up @@ -513,10 +513,6 @@ in
include ${config.services.nginx.package}/conf/fastcgi_params;
fastcgi_pass unix:/run/sympa/wwsympa.socket;
fastcgi_split_path_info ^(${loc})(.*)$;
fastcgi_param PATH_INFO $fastcgi_path_info;
fastcgi_param SCRIPT_FILENAME ${pkg}/lib/sympa/cgi/wwsympa.fcgi;
'';
}) // {
"/static-sympa/".alias = "${dataDir}/static_content/";
Expand Down

0 comments on commit 3f31c0d

Please sign in to comment.