Bug 54384 - Improve a confusing log message related to LimitRequestLine
Summary: Improve a confusing log message related to LimitRequestLine
Status: RESOLVED FIXED
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: Core (show other bugs)
Version: 2.4.3
Hardware: PC Linux
: P2 major (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL:
Keywords: FixedInTrunk, PatchAvailable
Depends on:
Blocks:
 
Reported: 2013-01-08 09:40 UTC by Jackie Zhang
Modified: 2013-05-12 10:28 UTC (History)
1 user (show)



Attachments
An improved log message patch to make the diagnosis easier (634 bytes, patch)
2013-01-08 09:40 UTC, Jackie Zhang
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jackie Zhang 2013-01-08 09:40:17 UTC
Created attachment 29828 [details]
An improved log message patch to make the diagnosis easier

Hi, all,

Is it possible to change the following log message to be more accurate and explicit that links to the configuration directive? (see the patch attached)

"AH00565: request failed: URI too long (longer than xxxx)"

Currently, it's kind of confusing without reading the code. I even do not know what the "URI" refers to when saw the log message: I didn't know whether it's a problem on the server side or client side.

Having read the code, I know this's because and only because the client's HTTP request-line exceeds the server's "LimitRequestLine" setting.

Compared with "LimitRequestLine", the log message of "LimitRequestFieldsize" is very very clear:

"AH00561: Request header exceeds LimitRequestFieldSize"

Moreover, the current message is not acurrate because request-line also consists of the HTTP method and protocol version besides URI.

I wonder whether we can have the same style of logging, which is implemented in the patch. 

Notice that the logging is at the place where "This is only going to happen if it exceeds the configured limit for a request-line", so the patch is valid. (You can search "HTTP_REQUEST_URI_TOO_LARGE" in the code to verify it) 

Thanks,
Jackie
Comment 1 Christophe JAILLET 2013-01-15 20:22:41 UTC
Thank you for the report.
I agree with your analysis.

This has been applied to trunk.
http://svn.apache.org/viewvc?view=revision&sortby=date&revision=1433613

If nobody complains, I will propose for backport for 2.4.x in the coming days.
Comment 2 Graham Leggett 2013-04-30 12:37:30 UTC
Proposed for backport to v2.4.
Comment 3 Graham Leggett 2013-05-12 10:28:24 UTC
Backported to v2.4.5.