nixos/oauth2_proxy_nginx: fix URL escaping
This commit is contained in:
parent
c8613033ca
commit
d85147ead0
|
@ -87,9 +87,9 @@ in
|
||||||
"/oauth2/auth" = let
|
"/oauth2/auth" = let
|
||||||
maybeQueryArg = name: value:
|
maybeQueryArg = name: value:
|
||||||
if value == null then null
|
if value == null then null
|
||||||
else "${name}=${lib.concatStringsSep "," value}";
|
else "${name}=${lib.concatStringsSep "," (builtins.map lib.escapeURL value)}";
|
||||||
allArgs = lib.mapAttrsToList maybeQueryArg conf;
|
allArgs = lib.mapAttrsToList maybeQueryArg conf;
|
||||||
cleanArgs = builtins.map lib.escapeURL (builtins.filter (x: x != null) allArgs);
|
cleanArgs = builtins.filter (x: x != null) allArgs;
|
||||||
cleanArgsStr = lib.concatStringsSep "&" cleanArgs;
|
cleanArgsStr = lib.concatStringsSep "&" cleanArgs;
|
||||||
in {
|
in {
|
||||||
# nginx doesn't support passing query string arguments to auth_request,
|
# nginx doesn't support passing query string arguments to auth_request,
|
||||||
|
|
Loading…
Reference in New Issue
Block a user