Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(221)

Issue 6877049: Cleanup http logs by handling broken pipe and connection reset by peer exceptions. (Closed)

Created:
9 years, 8 months ago by DaleCurtis
Modified:
9 years, 7 months ago
Reviewers:
Chris Masone, ericli
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli
Visibility:
Public.

Description

Cleanup http logs by handling broken pipe and connection reset by peer exceptions. New errors look like so: 04/19 12:16:04 ERROR|BaseHTTPSe:0447| localhost - - [19/Apr/2011 12:16:04] [Errno 32] Broken pipe 04/19 12:19:23 ERROR|BaseHTTPSe:0447| localhost - - [19/Apr/2011 12:19:23] [Errno 104] Connection reset by peer Change-Id: I767939f5f4bf84578b639fdf78321d88ed5afc88 BUG=none TEST=Started test server, forced resets/pipe failures. Ran bvt, smoke. Verified new messages are present. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=bd427e9

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -3 lines) Patch
M client/cros/httpd.py View 5 chunks +25 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
DaleCurtis
9 years, 8 months ago (2011-04-19 19:31:31 UTC) #1
Chris Masone
9 years, 8 months ago (2011-04-19 19:38:58 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698