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);