1. 06 Oct, 2016 1 commit
  2. 03 Oct, 2016 4 commits
  3. 01 Oct, 2016 3 commits
  4. 30 Sep, 2016 2 commits
    • Brad Fitzpatrick's avatar
      http2: in Server, disarm connection's ReadTimeout after first request · 69729301
      Brad Fitzpatrick authored
      Fix a regression from Go 1.5 to Go 1.6: when we introduced automatic
      HTTP/2 support in Go 1.6, we never handled Server.ReadTimeout. If a
      user set ReadTimeout, the net/http package would set the read deadline
      on the connection during the TLS handshake, but then the http2 package
      would never disarm it, killing the likely-still-in-use connection
      after the timeout period.
      
      This CL changes it to disarm the timeout after the first request
      headers, similar to net/http.Server.
      
      Unlike net/http.Server, we don't re-arm it on each idle period,
      because the definition of idle is more complicated with HTTP/2.
      
      No tests here for now. Tests will be in the go repo once all the knobs
      are in place and this is re-bundled into std, testing both http1 and
      http2.
      
      Updates golang/go#16450 (minimal fix for now)
      Updates golang/go#14204 (considering a new http1+http2 IdleTimeout knob)
      
      Change-Id: Iaa1570c118efda7dc0a65ba84cd77885699ec8fc
      Reviewed-on: https://go-review.googlesource.com/30077Reviewed-by: 's avatarIan Lance Taylor <iant@golang.org>
      Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      69729301
    • Brad Fitzpatrick's avatar
      http2: add Transport support for IdleConnTimeout · a333c534
      Brad Fitzpatrick authored
      Tests will be in the go repo's net/http package when this
      package is re-bundled into std.
      
      Updates golang/go#16808 (fixes after bundle into std)
      
      Change-Id: Iad31dc120bc008b1e9679bf7b2b988aac9c893c8
      Reviewed-on: https://go-review.googlesource.com/30075
      Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: 's avatarIan Lance Taylor <iant@golang.org>
      a333c534
  5. 29 Sep, 2016 4 commits
  6. 24 Sep, 2016 3 commits
  7. 21 Sep, 2016 1 commit
  8. 20 Sep, 2016 1 commit
  9. 16 Sep, 2016 1 commit
  10. 14 Sep, 2016 2 commits
  11. 12 Sep, 2016 3 commits
  12. 10 Sep, 2016 1 commit
  13. 07 Sep, 2016 1 commit
  14. 01 Sep, 2016 1 commit
  15. 31 Aug, 2016 1 commit
  16. 26 Aug, 2016 5 commits
  17. 25 Aug, 2016 1 commit
  18. 19 Aug, 2016 1 commit
    • Brad Fitzpatrick's avatar
      http2: fix protocol violation regression when writing certain request bodies · 7394c112
      Brad Fitzpatrick authored
      The mod_h2 workaround CL (git rev 28d1bd4f,
      https://golang.org/cl/25362) introduced a regression where the
      Transport could write two DATA frames, both with END_STREAM, if the
      Request.Body returned (non-0, io.EOF). strings.Reader, bytes.Reader
      are the most common Reader types used with HTTP requests and they only
      return (0, io.EOF) so we got generally lucky and things seemed to
      work, but other Readers do return (non-0, io.EOF), like the HTTP
      transport/server Readers. This is why we broke the HTTP proxy code,
      when proxying to HTTP/2.
      
      Updates golang/go#16788 (fixes after it's bundled into std)
      
      Change-Id: I42684017039eacfc27ee53e9c11431f713fdaca4
      Reviewed-on: https://go-review.googlesource.com/27406
      Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: 's avatarChris Broadfoot <cbro@golang.org>
      7394c112
  19. 11 Aug, 2016 1 commit
  20. 05 Aug, 2016 1 commit
  21. 04 Aug, 2016 1 commit
  22. 03 Aug, 2016 1 commit