- 20 Jun, 2016 1 commit
-
-
Matt Layher authored
Change-Id: I29487b0c1fdea97b5c35d22b4f7c23e11e7ceeed Reviewed-on: https://go-review.googlesource.com/24272Reviewed-by: David Anderson <danderson@google.com> Reviewed-by: Mikio Hara <mikioh.mikioh@gmail.com> Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com>
-
- 19 Jun, 2016 1 commit
-
-
Mikio Hara authored
Change-Id: I5b865483bf6a8bba07934ae3b1ff5bbbf6dd6a00 Reviewed-on: https://go-review.googlesource.com/24240Reviewed-by: Matt Layher <mdlayher@gmail.com> Reviewed-by: Tom Heng <zhm20070928@gmail.com> Reviewed-by: Michael Hudson-Doyle <michael.hudson@canonical.com>
-
- 18 Jun, 2016 1 commit
-
-
Matt Layher authored
Fixes golang/go#16055. Change-Id: I80437e2895b0f2bf23e090dec29bd20c2900db9a Reviewed-on: https://go-review.googlesource.com/24136Reviewed-by: David Anderson <danderson@google.com> Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com> Reviewed-by: Mikio Hara <mikioh.mikioh@gmail.com>
-
- 16 Jun, 2016 1 commit
-
-
Nigel Tao authored
Fixes golang/go#15967. Change-Id: I24ffebbe70d15cfd8697c843972f4f1669670f17 Reviewed-on: https://go-review.googlesource.com/24023Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 09 Jun, 2016 1 commit
-
-
Nigel Tao authored
Also follow the "go generate" convention, and have the generation command be a bare "go run gen.go" without any need for explicit flags. Stripping the for-debugging comments shrinks the generated table.go file from 538K to 139K. Also update table to latest list from publicsuffix.org: revision ea9f4f9c51f5bfc5c85a41a5637ab671c07c542a (2016-05-31T18:30:56Z) Change-Id: Ib59366106e0142fd65a2102e73d6a9a29e53930e Reviewed-on: https://go-review.googlesource.com/23930Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 08 Jun, 2016 1 commit
-
-
Mikio Hara authored
Fixes golang/go#15999. Change-Id: I20793ce717c768557c4942ff6be4e77c23ab201c Reviewed-on: https://go-review.googlesource.com/23880 Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 07 Jun, 2016 1 commit
-
-
Andrew Gerrand authored
Fixes golang/go#15930 Change-Id: Ib5d2f57361d52364edb29df25ec9a498c3088781 Reviewed-on: https://go-review.googlesource.com/23812Reviewed-by: Aaron Jacobs <jacobsa@google.com> Reviewed-by: Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org>
-
- 31 May, 2016 2 commits
-
-
Tom Bergan authored
Fixes golang/go#15777 Change-Id: I6fd8a8fdea051f3c02272cc15af7f0bcdfb5d745 Reviewed-on: https://go-review.googlesource.com/23526Reviewed-by: Mikio Hara <mikioh.mikioh@gmail.com> Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com>
-
Dave Cheney authored
The contract for Read specifies that a caller should process n bytes of data before handling an error value. The previous implementation may have discarded the final data read from the underlying reader if err was non nil. Change-Id: I81547cd9ea62c4060f79523679beed6bbc6ccd95 Reviewed-on: https://go-review.googlesource.com/23534Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 27 May, 2016 1 commit
-
-
H. İbrahim Güngör authored
WebDAV clients can't sort folders by date because 'getlastmodified' live property is not set. Fixes golang/go#15334. Change-Id: Ie56fcf8ae98173878e0972c0f9401151953334ff Reviewed-on: https://go-review.googlesource.com/23422Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 26 May, 2016 1 commit
-
-
Mikio Hara authored
Change-Id: Ib988bde8f8ff5719fd24bf63bd5ece648602dbd8 Reviewed-on: https://go-review.googlesource.com/23460 Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com> Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 25 May, 2016 1 commit
-
-
ayanamist authored
The existing transport implementation limit scheme to https, so simple h2c can not be used, if we loose the limit to http scheme, it works well with existing h2c server support. Fixes golang/go#15830 Change-Id: Icfe65d72b3e6fb41f69019539fb5b86144814378 Reviewed-on: https://go-review.googlesource.com/23181Reviewed-by: Andrew Gerrand <adg@golang.org> Run-TryBot: Andrew Gerrand <adg@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
- 21 May, 2016 1 commit
-
-
Brad Fitzpatrick authored
Since Go's HTTP/2 Handlers can read & write concurrently, it was previously possible for a Handler to get an HTTP Request with "Expect: 100-continue" and to first Write some data, then read some data, and the read of the data was triggering the server's automatic "100-continue" response path. But if the server had already replied with a non-100 HEADERS frame, sending a HEADERS frame with :status 100 later is in violation of the spec. Fix it by tracking whether we've already replied with something, and ignoring the 100-continue when it's inappropriate. Updates golang/go#14030 Change-Id: I6745ba0f257a31eaf63816e4263cadbef5a896a2 Reviewed-on: https://go-review.googlesource.com/23311Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 20 May, 2016 3 commits
-
-
Brad Fitzpatrick authored
Also, fix a Transport goroutine leak if the transport is still trying to write its body after the stream has completed. Updates golang/go#15425 (fixes after bundle into std) Change-Id: Id14d9360d012f53a963ec1999ef88fc592978b80 Reviewed-on: https://go-review.googlesource.com/23287 Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Brad Fitzpatrick authored
Go 1.6+ wires up its http.Transport values to http2 with ConfigureTransport. ConfigureTransport sets the connection pool to a "noDialClientConnPool" type. But the Transport.CloseIdleConnections implementation was specifically looking for a concrete type. Look for an interface instead. Updates golang/go#14607 (needs bundle into the main repo before fix) Change-Id: I6ad23b6facab5d3c2cbe71a1809a52794375d803 Reviewed-on: https://go-review.googlesource.com/23258 Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Andrew Gerrand <adg@golang.org>
-
Brad Fitzpatrick authored
In Go 1.6, the HTTP/1 client got Transport.ExpectContinueTimeout. This makes the HTTP/2 client respect a Request's "Expect: 100-continue" field and the Transport.ExpectContinueTimeout configuration. This also makes sure to call the traceWroteRequest hook if the server replied while we're still writing the request, since that code was in the same spot and it couldn't be trivially separated. Updates golang/go#13851 (fixed after integrating it into std) Updates golang/go#15744 Change-Id: I67dfd68532daa6c4a0c026549c6e5cbfce50e1ea Reviewed-on: https://go-review.googlesource.com/23235Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 19 May, 2016 2 commits
-
-
Brad Fitzpatrick authored
The http2.Transport was able to send bogus header keys & values. This changes rejects them earlier, before they hit the wire. In the process, mirror the lexical rules from the http package to x/net. Maintaining two copies has gotten increasingly annoying. Updates golang/go#14048 Change-Id: I20abcdeea92e7dc8706a1bbd60688ee8843a2b12 Reviewed-on: https://go-review.googlesource.com/23229Reviewed-by: Andrew Gerrand <adg@golang.org>
-
Brad Fitzpatrick authored
Updates golang/go#14188 Change-Id: Ic274841422fcb6179c0a782956bbfa336d27f1e1 Reviewed-on: https://go-review.googlesource.com/23230Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 18 May, 2016 5 commits
-
-
Brad Fitzpatrick authored
And act the same as HTTP/1.x in Go 1.7. Updates golang/go#15134 Change-Id: Ib64dd82cc5f8dd60e1680525f664d5b72be11fc6 Reviewed-on: https://go-review.googlesource.com/23220 Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Andrew Gerrand <adg@golang.org>
-
Brad Fitzpatrick authored
Updates golang/go#12580 Change-Id: I95d7206a8fde494f88288b381814a5d72d42df5e Reviewed-on: https://go-review.googlesource.com/23205 Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Mikio Hara authored
Change-Id: Ia431a6cfc53a66b20307f1e54b17f5bbceee02e3 Reviewed-on: https://go-review.googlesource.com/23193 Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Mikio Hara authored
Change-Id: I6789d04215f23dfb9fe07a90f78e80eadfcb906c Reviewed-on: https://go-review.googlesource.com/23192 Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Mikio Hara authored
Change-Id: Id0558e3ee71b393e0366c1ae5e022d657fec08f0 Reviewed-on: https://go-review.googlesource.com/23191Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 15 May, 2016 2 commits
-
-
David Anderson authored
Fixes golang/go#14974 Change-Id: I58c41acf29329aedf61b9ca59eb271e4536c80ea Reviewed-on: https://go-review.googlesource.com/23107Reviewed-by: Mikio Hara <mikioh.mikioh@gmail.com>
-
David Anderson authored
Updates golang/go#14974 Change-Id: I80da378a788d5d826b3b79ab308372231c012391 Reviewed-on: https://go-review.googlesource.com/23105Reviewed-by: Mikio Hara <mikioh.mikioh@gmail.com> Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
- 14 May, 2016 1 commit
-
-
Mikio Hara authored
This change introduces a package that provides the basic manipulation of routing facilities on BSD variants. Unlike the existing APIs in syscall package of standard library, the package tries to provide operating system and its architecture agnostic APIs. At present, the package supports any version of Darwin, any version of DragonFly BSD, FreeBSD 7 through 11, NetBSD 6 and above, and OpenBSD 5.6 and above. Updates golang/go#14724. Change-Id: Id964ea22dec491ddac3776e3a8c8c10f140f96ac Reviewed-on: https://go-review.googlesource.com/22446 Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 13 May, 2016 1 commit
-
-
Carl Mastrangelo authored
This change fixes a few bugs in the HPACK decoder: * Excess trailing padding is treated as an error per the HPACK Spec section 5.2 * Non EOS prefix padding is treated as an error * Max length is now enforced for all decoded symbols The idea here is to keep track of the decoded symbol length, rather than the number of unconsumed bits in cur. To this end, nbits has been renamed cbits (cur bits), and sbits (sym bits) has been introduced. The main problem with using nbits is that it can easily be zero, such as when decoding {0xff, 0xff}. Using a clear moniker makes it easier to see why checking cbits > 0 at the end of the function is incorrect. Fixes golang/go#15614 Change-Id: I1ae868caa9c207fcf9c9dec7f10ee9f400211f99 Reviewed-on: https://go-review.googlesource.com/23067Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 10 May, 2016 1 commit
-
-
Brad Fitzpatrick authored
Go 1.7 introduces Request.Context and Response.Uncompressed. Use them both in the http2 Transport when building for Go 1.7. Tests are in the main repo. Updates golang/go#15366 Updates golang/go#15134 Change-Id: I4b78a2710b6cf30c769c7f85cea15ecde653a317 Reviewed-on: https://go-review.googlesource.com/23002Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 06 May, 2016 1 commit
-
-
Ian Lance Taylor authored
Change-Id: I62bfc730643cb0756fc67f3c5cf73029b75c07bb Reviewed-on: https://go-review.googlesource.com/22884Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 05 May, 2016 2 commits
-
-
Mikio Hara authored
Change-Id: I7fc0634dd1c7cd49b0f3aa2d84a83e783f6687a1 Reviewed-on: https://go-review.googlesource.com/22678Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Mikio Hara authored
Change-Id: I5ceede6cf0aa4fcdbf59c39e1aff9826874f2710 Reviewed-on: https://go-review.googlesource.com/22677Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 01 May, 2016 1 commit
-
-
Nic Day authored
Abbreviating "if and only if" to "iff" is against Go style in public documentation; use expanded form. Change-Id: I23afdca1b773ef9572794555dfb768c9ded90ab7 Reviewed-on: https://go-review.googlesource.com/22545Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 27 Apr, 2016 1 commit
-
-
Brad Fitzpatrick authored
Fixes golang/go#15449 Change-Id: Ib4802dc1d9208abc17ef31212d49f03f4f4ed3d1 Reviewed-on: https://go-review.googlesource.com/22494Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 26 Apr, 2016 1 commit
-
-
Brad Fitzpatrick authored
But still check that the high bit isn't set. Fixes golang/go#15444 Change-Id: Ic9339244d3d0c4e4d6573a691b3ba37c9eac3981 Reviewed-on: https://go-review.googlesource.com/22476Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 22 Apr, 2016 1 commit
-
-
Dave Day authored
This was always a no-op in 1.4 and earlier. This change directly inlines the 1.5 version. Change-Id: I6dcf69497d876d7dde729c1801cef5382b97fa07 Reviewed-on: https://go-review.googlesource.com/22388Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 21 Apr, 2016 1 commit
-
-
Dave Day authored
The Go 1.4 (and earlier) support for request cancellation is racy and complicates the implementation of ctxhttp. This change simply inlines the existing 1.5+ cancellation logic directly, and removes support for older versions. Change-Id: I7df9f0648dd0e571366374d079e2976050464ca3 Reviewed-on: https://go-review.googlesource.com/22302Reviewed-by: David Symonds <dsymonds@golang.org> Run-TryBot: David Symonds <dsymonds@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 12 Apr, 2016 2 commits
-
-
David Symonds authored
9d8ef8d7 caused an extra allocation for gccgo. Change-Id: I6568d84009882103ec4767546599afa20c54c480 Reviewed-on: https://go-review.googlesource.com/21940Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Dan Peterson authored
Updates golang/go#15258 Change-Id: Ieaea9564a6e300938a75d3382151c46e9bde1e59 Reviewed-on: https://go-review.googlesource.com/21903Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 11 Apr, 2016 2 commits
-
-
Mikio Hara authored
Change-Id: I4ce6dfec994252fec3874fb748e0d6b8db97394d Reviewed-on: https://go-review.googlesource.com/21835Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Mikio Hara authored
Change-Id: I1e4133d1988cb745d465b478bcdf01594304e48a Reviewed-on: https://go-review.googlesource.com/21834Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-