Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(288)

Issue 4235056: code review 4235056: http: add test for recently fixed HTTP/1.0 keep-alive issue (Closed)

Can't Edit
Can't Publish+Mail
Created:
11 years, 5 months ago by adg
Modified:
11 years, 5 months ago
Reviewers:
CC:
bradfitz, rsc, bradfitzgoog, golang-dev
Visibility:
Public.

Description

http: add test for recently fixed HTTP/1.0 keep-alive issue

Patch Set 1 #

Patch Set 2 : diff -r 5ccb9b6f8cd1 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -0 lines) Patch

Messages

Total messages: 4
adg
Hello bradfitzgo, rsc (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://go.googlecode.com/hg/
11 years, 5 months ago (2011-03-07 06:01:54 UTC) #1
rsc
LGTM
11 years, 5 months ago (2011-03-07 12:49:29 UTC) #2
bradfitzgoog
LGTM Thanks! But there are now at least 2 tests that do this pattern (of ...
11 years, 5 months ago (2011-03-07 16:20:44 UTC) #3
adg
11 years, 5 months ago (2011-03-08 05:12:51 UTC) #4
*** Submitted as http://code.google.com/p/go/source/detail?r=ea80f54d6038 ***

http: add test for recently fixed HTTP/1.0 keep-alive issue

R=bradfitzgo, rsc, bradfitzwork
CC=golang-dev
http://codereview.appspot.com/4235056

Committer: Andrew Gerrand <adg@golang.org>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b