| package http_test |
| |
| import ( |
| "errors" |
| "fmt" |
| "io" |
| . "net/http" |
| "os" |
| "sync" |
| "testing" |
| "time" |
| ) |
| |
| func TestResponseControllerFlush(t *testing.T) { run(t, testResponseControllerFlush) } |
| func testResponseControllerFlush(t *testing.T, mode testMode) { |
| continuec := make(chan struct{}) |
| cst := newClientServerTest(t, mode, HandlerFunc(func(w ResponseWriter, r *Request) { |
| ctl := NewResponseController(w) |
| w.Write([]byte("one")) |
| if err := ctl.Flush(); err != nil { |
| t.Errorf("ctl.Flush() = %v, want nil", err) |
| return |
| } |
| <-continuec |
| w.Write([]byte("two")) |
| })) |
| |
| res, err := cst.c.Get(cst.ts.URL) |
| if err != nil { |
| t.Fatalf("unexpected connection error: %v", err) |
| } |
| defer res.Body.Close() |
| |
| buf := make([]byte, 16) |
| n, err := res.Body.Read(buf) |
| close(continuec) |
| if err != nil || string(buf[:n]) != "one" { |
| t.Fatalf("Body.Read = %q, %v, want %q, nil", string(buf[:n]), err, "one") |
| } |
| |
| got, err := io.ReadAll(res.Body) |
| if err != nil || string(got) != "two" { |
| t.Fatalf("Body.Read = %q, %v, want %q, nil", string(got), err, "two") |
| } |
| } |
| |
| func TestResponseControllerHijack(t *testing.T) { run(t, testResponseControllerHijack) } |
| func testResponseControllerHijack(t *testing.T, mode testMode) { |
| const header = "X-Header" |
| const value = "set" |
| cst := newClientServerTest(t, mode, HandlerFunc(func(w ResponseWriter, r *Request) { |
| ctl := NewResponseController(w) |
| c, _, err := ctl.Hijack() |
| if mode == http2Mode { |
| if err == nil { |
| t.Errorf("ctl.Hijack = nil, want error") |
| } |
| w.Header().Set(header, value) |
| return |
| } |
| if err != nil { |
| t.Errorf("ctl.Hijack = _, _, %v, want _, _, nil", err) |
| return |
| } |
| fmt.Fprintf(c, "HTTP/1.0 200 OK\r\n%v: %v\r\nContent-Length: 0\r\n\r\n", header, value) |
| })) |
| res, err := cst.c.Get(cst.ts.URL) |
| if err != nil { |
| t.Fatal(err) |
| } |
| if got, want := res.Header.Get(header), value; got != want { |
| t.Errorf("response header %q = %q, want %q", header, got, want) |
| } |
| } |
| |
| func TestResponseControllerSetPastWriteDeadline(t *testing.T) { |
| run(t, testResponseControllerSetPastWriteDeadline) |
| } |
| func testResponseControllerSetPastWriteDeadline(t *testing.T, mode testMode) { |
| cst := newClientServerTest(t, mode, HandlerFunc(func(w ResponseWriter, r *Request) { |
| ctl := NewResponseController(w) |
| w.Write([]byte("one")) |
| if err := ctl.Flush(); err != nil { |
| t.Errorf("before setting deadline: ctl.Flush() = %v, want nil", err) |
| } |
| if err := ctl.SetWriteDeadline(time.Now().Add(-10 * time.Second)); err != nil { |
| t.Errorf("ctl.SetWriteDeadline() = %v, want nil", err) |
| } |
| |
| w.Write([]byte("two")) |
| if err := ctl.Flush(); err == nil { |
| t.Errorf("after setting deadline: ctl.Flush() = nil, want non-nil") |
| } |
| // Connection errors are sticky, so resetting the deadline does not permit |
| // making more progress. We might want to change this in the future, but verify |
| // the current behavior for now. If we do change this, we'll want to make sure |
| // to do so only for writing the response body, not headers. |
| if err := ctl.SetWriteDeadline(time.Now().Add(1 * time.Hour)); err != nil { |
| t.Errorf("ctl.SetWriteDeadline() = %v, want nil", err) |
| } |
| w.Write([]byte("three")) |
| if err := ctl.Flush(); err == nil { |
| t.Errorf("after resetting deadline: ctl.Flush() = nil, want non-nil") |
| } |
| })) |
| |
| res, err := cst.c.Get(cst.ts.URL) |
| if err != nil { |
| t.Fatalf("unexpected connection error: %v", err) |
| } |
| defer res.Body.Close() |
| b, _ := io.ReadAll(res.Body) |
| if string(b) != "one" { |
| t.Errorf("unexpected body: %q", string(b)) |
| } |
| } |
| |
| func TestResponseControllerSetFutureWriteDeadline(t *testing.T) { |
| run(t, testResponseControllerSetFutureWriteDeadline) |
| } |
| func testResponseControllerSetFutureWriteDeadline(t *testing.T, mode testMode) { |
| errc := make(chan error, 1) |
| startwritec := make(chan struct{}) |
| cst := newClientServerTest(t, mode, HandlerFunc(func(w ResponseWriter, r *Request) { |
| ctl := NewResponseController(w) |
| w.WriteHeader(200) |
| if err := ctl.Flush(); err != nil { |
| t.Errorf("ctl.Flush() = %v, want nil", err) |
| } |
| <-startwritec // don't set the deadline until the client reads response headers |
| if err := ctl.SetWriteDeadline(time.Now().Add(1 * time.Millisecond)); err != nil { |
| t.Errorf("ctl.SetWriteDeadline() = %v, want nil", err) |
| } |
| _, err := io.Copy(w, neverEnding('a')) |
| errc <- err |
| })) |
| |
| res, err := cst.c.Get(cst.ts.URL) |
| close(startwritec) |
| if err != nil { |
| t.Fatalf("unexpected connection error: %v", err) |
| } |
| defer res.Body.Close() |
| _, err = io.Copy(io.Discard, res.Body) |
| if err == nil { |
| t.Errorf("client reading from truncated request body: got nil error, want non-nil") |
| } |
| err = <-errc // io.Copy error |
| if !errors.Is(err, os.ErrDeadlineExceeded) { |
| t.Errorf("server timed out writing request body: got err %v; want os.ErrDeadlineExceeded", err) |
| } |
| } |
| |
| func TestResponseControllerSetPastReadDeadline(t *testing.T) { |
| run(t, testResponseControllerSetPastReadDeadline) |
| } |
| func testResponseControllerSetPastReadDeadline(t *testing.T, mode testMode) { |
| readc := make(chan struct{}) |
| cst := newClientServerTest(t, mode, HandlerFunc(func(w ResponseWriter, r *Request) { |
| ctl := NewResponseController(w) |
| b := make([]byte, 3) |
| n, err := io.ReadFull(r.Body, b) |
| b = b[:n] |
| if err != nil || string(b) != "one" { |
| t.Errorf("before setting read deadline: Read = %v, %q, want nil, %q", err, string(b), "one") |
| return |
| } |
| if err := ctl.SetReadDeadline(time.Now()); err != nil { |
| t.Errorf("ctl.SetReadDeadline() = %v, want nil", err) |
| return |
| } |
| b, err = io.ReadAll(r.Body) |
| if err == nil || string(b) != "" { |
| t.Errorf("after setting read deadline: Read = %q, nil, want error", string(b)) |
| } |
| close(readc) |
| // Connection errors are sticky, so resetting the deadline does not permit |
| // making more progress. We might want to change this in the future, but verify |
| // the current behavior for now. |
| if err := ctl.SetReadDeadline(time.Time{}); err != nil { |
| t.Errorf("ctl.SetReadDeadline() = %v, want nil", err) |
| return |
| } |
| b, err = io.ReadAll(r.Body) |
| if err == nil { |
| t.Errorf("after resetting read deadline: Read = %q, nil, want error", string(b)) |
| } |
| })) |
| |
| pr, pw := io.Pipe() |
| var wg sync.WaitGroup |
| wg.Add(1) |
| go func() { |
| defer wg.Done() |
| pw.Write([]byte("one")) |
| <-readc |
| pw.Write([]byte("two")) |
| pw.Close() |
| }() |
| defer wg.Wait() |
| res, err := cst.c.Post(cst.ts.URL, "text/foo", pr) |
| if err == nil { |
| defer res.Body.Close() |
| } |
| } |
| |
| func TestResponseControllerSetFutureReadDeadline(t *testing.T) { |
| run(t, testResponseControllerSetFutureReadDeadline) |
| } |
| func testResponseControllerSetFutureReadDeadline(t *testing.T, mode testMode) { |
| respBody := "response body" |
| cst := newClientServerTest(t, mode, HandlerFunc(func(w ResponseWriter, req *Request) { |
| ctl := NewResponseController(w) |
| if err := ctl.SetReadDeadline(time.Now().Add(1 * time.Millisecond)); err != nil { |
| t.Errorf("ctl.SetReadDeadline() = %v, want nil", err) |
| } |
| _, err := io.Copy(io.Discard, req.Body) |
| if !errors.Is(err, os.ErrDeadlineExceeded) { |
| t.Errorf("server timed out reading request body: got err %v; want os.ErrDeadlineExceeded", err) |
| } |
| w.Write([]byte(respBody)) |
| })) |
| pr, pw := io.Pipe() |
| res, err := cst.c.Post(cst.ts.URL, "text/apocryphal", pr) |
| if err != nil { |
| t.Fatal(err) |
| } |
| defer res.Body.Close() |
| got, err := io.ReadAll(res.Body) |
| if string(got) != respBody || err != nil { |
| t.Errorf("client read response body: %q, %v; want %q, nil", string(got), err, respBody) |
| } |
| pw.Close() |
| } |
| |
| type wrapWriter struct { |
| ResponseWriter |
| } |
| |
| func (w wrapWriter) Unwrap() ResponseWriter { |
| return w.ResponseWriter |
| } |
| |
| func TestWrappedResponseController(t *testing.T) { run(t, testWrappedResponseController) } |
| func testWrappedResponseController(t *testing.T, mode testMode) { |
| cst := newClientServerTest(t, mode, HandlerFunc(func(w ResponseWriter, r *Request) { |
| w = wrapWriter{w} |
| ctl := NewResponseController(w) |
| if err := ctl.Flush(); err != nil { |
| t.Errorf("ctl.Flush() = %v, want nil", err) |
| } |
| if err := ctl.SetReadDeadline(time.Time{}); err != nil { |
| t.Errorf("ctl.SetReadDeadline() = %v, want nil", err) |
| } |
| if err := ctl.SetWriteDeadline(time.Time{}); err != nil { |
| t.Errorf("ctl.SetWriteDeadline() = %v, want nil", err) |
| } |
| })) |
| res, err := cst.c.Get(cst.ts.URL) |
| if err != nil { |
| t.Fatalf("unexpected connection error: %v", err) |
| } |
| io.Copy(io.Discard, res.Body) |
| defer res.Body.Close() |
| } |