Return-Path: Delivered-To: apmail-jakarta-httpcomponents-dev-archive@www.apache.org Received: (qmail 49083 invoked from network); 29 Nov 2006 13:56:59 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 29 Nov 2006 13:56:59 -0000 Received: (qmail 93087 invoked by uid 500); 29 Nov 2006 13:56:53 -0000 Delivered-To: apmail-jakarta-httpcomponents-dev-archive@jakarta.apache.org Received: (qmail 93016 invoked by uid 500); 29 Nov 2006 13:56:53 -0000 Mailing-List: contact httpcomponents-dev-help@jakarta.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "HttpClient Project" Delivered-To: mailing list httpcomponents-dev@jakarta.apache.org Received: (qmail 92985 invoked by uid 99); 29 Nov 2006 13:56:53 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 29 Nov 2006 05:56:53 -0800 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO brutus.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 29 Nov 2006 05:56:42 -0800 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 5D7FC7142FA for ; Wed, 29 Nov 2006 05:56:22 -0800 (PST) Message-ID: <14529564.1164808582380.JavaMail.jira@brutus> Date: Wed, 29 Nov 2006 05:56:22 -0800 (PST) From: =?utf-8?Q?Endre_St=C3=B8lsvik_=28JIRA=29?= To: httpcomponents-dev@jakarta.apache.org Subject: [jira] Commented: (HTTPCLIENT-609) Use TRACE logging instead of DEBUG for the absolute nitty-gritties In-Reply-To: <15866867.1164722541051.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Virus-Checked: Checked by ClamAV on apache.org [ http://issues.apache.org/jira/browse/HTTPCLIENT-609?page=3Dcomments#a= ction_12454311 ]=20 =20 Endre St=C3=B8lsvik commented on HTTPCLIENT-609: ------------------------------------------- @Roland: I don't quite see what you're answering/commenting - did you read = the entire description?! I did NOT at ANY POINT say that you should stop using DEBUG: I've however n= ot seen a single log-line from httpclient that outputs on TRACE. (Full wire= -dump doesn't belong in DEBUG, IM(F)O) In my description, I try to point out that I'd appreciate IF you DID use TR= ACE "somewhat more extensively" (given that it isn't used at all now, AFAIC= S). And I came up with some quick candidates for which loglines that would be b= etter sent to TRACE than DEBUG. Given that I do not suggest that you delete one single log-statement, only = "downgrade" some loglines from DEBUG to TRACE, I do not understand where yo= u're coming from in your comment at all. Also, I firmly believe that you folks should think of "the developers" not = as yourself, but as the users of your library. There are many more of the l= atter than the former. However, given, again, that I do not suggest that yo= u delete one single logline, I can't seem to get why not both camps could b= e pleased in this particular situation. > Use TRACE logging instead of DEBUG for the absolute nitty-gritties > ------------------------------------------------------------------ > > Key: HTTPCLIENT-609 > URL: http://issues.apache.org/jira/browse/HTTPCLIENT-609 > Project: HttpComponents HttpClient > Issue Type: Improvement > Affects Versions: 3.1 Beta 1 > Environment: n/a > Reporter: Endre St=C3=B8lsvik > Priority: Minor > Fix For: 4.0 Alpha 1 > > > [This is basically a copy of the Spring improvement request SPR-2873: htt= p://opensource.atlassian.com/projects/spring/browse/SPR-2873 ) > Given a developer situation: Much of the DEBUG information in the log of = HttpClient is very un-interesting as long as it works. Some of these lines = are however of much bigger importance than others (thus turning off DEBUG g= lobally for HttpClient isn't good either). > TRACE and DEBUG are the two developer-centric logging levels of log4j and= commons logging (the rest are "production levels"). Since log4j-1.2.12, TR= ACE have existed. Clogging have always had trace, but before release 1.1 ma= pped Log.trace to log4j's DEBUG, but 1.1 (released May 9. 2006) now maps to= log4j's TRACE. > I think that HttpClient's logging would benefit a lot by using TRACE leve= l extensively, in that developers could turn all of httpclient's logging do= wn to DEBUG, but still see "major developer events" like connections being = opened, the request being sent, and e.g. the response's status line, size o= f headers and body, keep-alive vs. closing of connection. > Candidates for TRACE level include: > * httpclient.wire.* > * org.apache.commons.httpclient.params.DefaultHttpParams > * org.apache.commons.httpclient.HttpMethodBase > * .. and probably a bunch of others that doesn't bring the developer in= the standard "good flow mode" any highly interesting information.=20 > Please note that I do NOT view these lines as worthless. It is however in= _normal_ developer circumstances not valuable information, and it would ea= se development if it was possible to turn these ultra-verbose loglines off = easily. When things just aren't working out, and your exciting REST-based q= uery doesn't work out, or your charset encodings just doesn't give what you= 're expecting, you'd turn on TRACE to really get down to the hard core. You= 'd find the problem, fix it, and set it to DEBUG again. > In addition, the lines that were left on the DEBUG level should obviously= be as informative as possible, and thus maybe somewhat more verbose than n= ow, trying to "aggregate" some pieces of information that now are output ov= er several DEBUG lines.. > I do realize that I could achive a lot of this with a rather extensive lo= g configuration, that also had to include raw text filters, but I do believ= e that this affects more developers than me! > PS: it wouldn't hurt either if all of httpclient's log-lines came from a = common root, e.g. "HttpClient", or "org.apache.commons.httpclient", instead= of having several roots. This would however be a somewhat "backward incomp= atible" change, since it now has (at least?) two roots. --=20 This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: htt= p://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: httpcomponents-dev-unsubscribe@jakarta.apache.org For additional commands, e-mail: httpcomponents-dev-help@jakarta.apache.org