tree 409548c5784bcd6ab31c82cbfd032df7009baa58
parent 039a4258aec0ad3c79b905677cceeab13b296a77
author Brad Fitzpatrick <bradfitz@golang.org> 1531353487 +0000
committer Brad Fitzpatrick <bradfitz@golang.org> 1531371573 +0000

http2: use GetBody unconditionally on Transport retry, when available

We were previously only using the new-ish Request.GetBody to "rewind"
a Request.Body on retry when it seemed that we hadn't started the
read+write body copy process from the old request yet.

Apparently there's a bug somewhere, so this is a safe minimal fix for
now, unconditionally using GetBody when it's available, rather than
only using it when it seems we need to. Should have no performance impact
because it's supposed to be cheap, and this only happens on rare retries
where the server's GOAWAY came in-flight while we were writing a request.

Updates golang/go#25009 (not a fix, but enough for Go 1.11)

Change-Id: Ia462944d4a68cf2fde8d32b7b357b450c509a349
Reviewed-on: https://go-review.googlesource.com/123476
Reviewed-by: Ian Lance Taylor <iant@golang.org>