You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by co...@apache.org on 2019/08/08 21:23:29 UTC

svn commit: r1864735 - /httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c

Author: covener
Date: Thu Aug  8 21:23:29 2019
New Revision: 1864735

URL: http://svn.apache.org/viewvc?rev=1864735&view=rev
Log:
followup to r1864734

Modified:
    httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c

Modified: httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c?rev=1864735&r1=1864734&r2=1864735&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c (original)
+++ httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c Thu Aug  8 21:23:29 2019
@@ -783,7 +783,7 @@ apr_status_t h2_stream_end_headers(h2_st
         apr_table_do(table_check_val_len, &ctx, stream->request->headers, NULL);
         if (ctx.failed_key) {
             ap_log_cerror(APLOG_MARK, APLOG_INFO, 0, stream->session->c,  
-                          H2_STRM_LOG(APLOGNO(10190) stream,"Request header exceeds "
+                          H2_STRM_LOG(APLOGNO(10190), stream,"Request header exceeds "
                                       "LimitRequestFieldSize: %.*s"),
                           (int)H2MIN(strlen(ctx.failed_key), 80), ctx.failed_key);
             set_error_response(stream, HTTP_REQUEST_HEADER_FIELDS_TOO_LARGE);



Re: svn commit: r1864735 - /httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c

Posted by Marion & Christophe JAILLET <ch...@wanadoo.fr>.
Thx for spotting and fixing it Eric.

The comma has been automagically removed by 'make update-log-msg-tags'
Will have a look to see if i tweak the script in order to avoid that in 
the future.

CJ


Le 08/08/2019 à 23:23, covener@apache.org a écrit :
> Author: covener
> Date: Thu Aug  8 21:23:29 2019
> New Revision: 1864735
>
> URL: http://svn.apache.org/viewvc?rev=1864735&view=rev
> Log:
> followup to r1864734
>
> Modified:
>      httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c
>
> Modified: httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c
> URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c?rev=1864735&r1=1864734&r2=1864735&view=diff
> ==============================================================================
> --- httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c (original)
> +++ httpd/httpd/branches/2.4.x/modules/http2/h2_stream.c Thu Aug  8 21:23:29 2019
> @@ -783,7 +783,7 @@ apr_status_t h2_stream_end_headers(h2_st
>           apr_table_do(table_check_val_len, &ctx, stream->request->headers, NULL);
>           if (ctx.failed_key) {
>               ap_log_cerror(APLOG_MARK, APLOG_INFO, 0, stream->session->c,
> -                          H2_STRM_LOG(APLOGNO(10190) stream,"Request header exceeds "
> +                          H2_STRM_LOG(APLOGNO(10190), stream,"Request header exceeds "
>                                         "LimitRequestFieldSize: %.*s"),
>                             (int)H2MIN(strlen(ctx.failed_key), 80), ctx.failed_key);
>               set_error_response(stream, HTTP_REQUEST_HEADER_FIELDS_TOO_LARGE);
>
>
>