212440 - nsHTTP:2 shouldn't include traffic

Status: RESOLVED FIXED

(Core :: Networking: HTTP, defect)

Description

[timeless]

I'm crashing a lot on BeOS so i decided to do:
NSPR_LOG_MODULES=all:2 ./mozilla

when people need all output they use all:5
here are the constants:
 135 typedef enum PRLogModuleLevel {
 136     PR_LOG_NONE = 0,                /* nothing */
 137     PR_LOG_ALWAYS = 1,              /* always printed */
 138     PR_LOG_ERROR = 2,               /* error messages */
 139     PR_LOG_WARNING = 3,             /* warning messages */
 140     PR_LOG_DEBUG = 4,               /* debug messages */
 141
 142     PR_LOG_NOTICE = PR_LOG_DEBUG,   /* notice messages */
 143     PR_LOG_WARN = PR_LOG_WARNING,   /* warning messages */

the http logging
-2146344064[80116380]: http response [
-2146344064[80116380]:   HTTP/1.1 200 OK

imo qualifies as debug messages. it isn't critical and it isn't an error.
I'll settle for warning. i just need all:2 to have much less content.

Comment 1

[timeless]

Attachment: Require nsHTTP:4 to get traffic

Comment 2

[Darin Fisher]

the point was so that folks can see just the headers instead of headers mixed
with all the other traffic.  LOG4 is all traffic in HTTP land, so please use
LOG3 instead.  also, i would appreciate if you would update this document:

   http://www.mozilla.org/projects/netlib/http/http-debugging.html

thx

Comment 3

[timeless]

nsHttp:3 it is. fixed in cvs and on gila