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
  • Loading branch information
mmilata committed Jun 10, 2020
1 parent 511f062 commit bd4e4dd
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions nixos/modules/services/mail/sympa.nix
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 bd4e4dd

Please sign in to comment.