diff options
author | Graham Dumpleton <Graham.Dumpleton@gmail.com> | 2022-08-18 22:22:51 +1000 |
---|---|---|
committer | Graham Dumpleton <Graham.Dumpleton@gmail.com> | 2022-08-18 22:22:51 +1000 |
commit | 3af5a868ad7f3becc52bd8e2acdb7bfd8bfc3a0c (patch) | |
tree | b9ec2de6bad09cf229ca72632c74c404d59dae9c | |
parent | 1ac5192470aec18883fef28dca09423ea20f767b (diff) | |
download | mod_wsgi-3af5a868ad7f3becc52bd8e2acdb7bfd8bfc3a0c.tar.gz |
Add more details to comment about forcing unlimited value for request content for daemon.
-rw-r--r-- | src/server/mod_wsgi.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/server/mod_wsgi.c b/src/server/mod_wsgi.c index 4ae2eb4..3f5d855 100644 --- a/src/server/mod_wsgi.c +++ b/src/server/mod_wsgi.c @@ -12909,8 +12909,13 @@ static int wsgi_hook_daemon_handler(conn_rec *c) * Try and ensure that request body limit in daemon mode process * is unlimited as Apache 2.4.54 changed rules for limit and if * unset is now overridden by HTTP filters to be 1GiB rather than - * unlimited. Not sure if this is required or not but could be - * since we populate configuration from base server config only. + * unlimited. This is required since we populate configuration + * from the base server config only so setting unlimited in a more + * specific context such as a virtual host wouldn't be visible. + * Note that setting this to unlimited in the daemon mode process + * is okay as the request limit body is checked in the Apache + * child process before request is proxied specifically to avoid + * unecessarily passing the content across to the daemon process. */ d = (core_dir_config *)ap_get_core_module_config(r->per_dir_config); |