You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by yl...@apache.org on 2018/04/20 10:38:05 UTC
svn commit: r1829643 - /httpd/httpd/trunk/modules/http/http_filters.c
Author: ylavic
Date: Fri Apr 20 10:38:05 2018
New Revision: 1829643
URL: http://svn.apache.org/viewvc?rev=1829643&view=rev
Log:
Restore comment about LimitRequestBody vs proxied responses.
ResponseFieldSize actually applies to headers (not body), so the comment was
valid but did match the code (addressed by r1829642 still).
Modified:
httpd/httpd/trunk/modules/http/http_filters.c
Modified: httpd/httpd/trunk/modules/http/http_filters.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http/http_filters.c?rev=1829643&r1=1829642&r2=1829643&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/http/http_filters.c (original)
+++ httpd/httpd/trunk/modules/http/http_filters.c Fri Apr 20 10:38:05 2018
@@ -317,9 +317,10 @@ apr_status_t ap_http_filter(ap_filter_t
f->ctx = ctx = apr_pcalloc(f->r->pool, sizeof(*ctx));
ctx->state = BODY_NONE;
- /* LimitRequestBody does not apply to proxied responses, which have
- * their own ResponseFieldSize parameter.
+ /* LimitRequestBody does not apply to proxied responses.
* Consider implementing this check in its own filter.
+ * Would adding a directive to limit the size of proxied
+ * responses be useful?
*/
if (f->r->proxyreq != PROXYREQ_RESPONSE) {
ctx->limit = ap_get_limit_req_body(f->r);