- 03 Jun, 2015 1 commit
-
-
Nigel Tao authored
Fixes golang/go#10881 Change-Id: I9f16dd0425d4e3c4a05530411eb29a7193a9d242 Reviewed-on: https://go-review.googlesource.com/10621Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 01 Jun, 2015 1 commit
-
-
Nigel Tao authored
Change-Id: I78c9a2e06da5269140212996842ac2e885d2746c Reviewed-on: https://go-review.googlesource.com/10561Reviewed-by: David Symonds <dsymonds@golang.org>
-
- 29 May, 2015 1 commit
-
-
Nigel Tao authored
A DeadPropsHolder could be backed by disk, whether directly or indirectly via a database, so we should allow for I/O to fail. Change-Id: Id40bcc86eb854212ef254ea1e1c54fd2a2b1960a Reviewed-on: https://go-review.googlesource.com/10472Reviewed-by: Robert Stepanek <robert.stepanek@gmail.com> Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 28 May, 2015 2 commits
-
-
Nigel Tao authored
Change-Id: I778e26c06f153fb705b450052d3721fec9a3082b Reviewed-on: https://go-review.googlesource.com/10471Reviewed-by: Robert Stepanek <robert.stepanek@gmail.com> Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
Robert Stepanek authored
POST and PUT requests. Change-Id: Id7b8a0aff7af1edefc45e2441565a2261f539895 Reviewed-on: https://go-review.googlesource.com/10395Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 27 May, 2015 3 commits
-
-
Robert Stepanek authored
webdav.Dir resource. Change-Id: If28dc1853cc3f393a53bf8f0e0f0bb87328c3d29 Reviewed-on: https://go-review.googlesource.com/10394Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
Nigel Tao authored
Change-Id: I38ac4a507cb79d4295c58f2d8c891e5bdcf0c1e4 Reviewed-on: https://go-review.googlesource.com/10379Reviewed-by: Robert Stepanek <robert.stepanek@gmail.com> Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
Mikio Hara authored
Fixes golang/go#10951. Change-Id: I94bf948ce74f8289008930701b2825ffcf57fce1 Reviewed-on: https://go-review.googlesource.com/10378Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 22 May, 2015 2 commits
-
-
Mikio Hara authored
This change splits the existing ping test into non-privileged and privileged tests to cover IPConn full stack test on behalf of the standard library. Updates golang/go#10730. Change-Id: I5d2e00c0b42b857045414eb8e0efca393967742e Reviewed-on: https://go-review.googlesource.com/10090Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Nigel Tao authored
Change-Id: I569993723942f71599411a25ff31e97c1bc8875c Reviewed-on: https://go-review.googlesource.com/10305Reviewed-by: Robert Stepanek <robert.stepanek@gmail.com> Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 21 May, 2015 1 commit
-
-
Nigel Tao authored
Before/after: <- summary for `props': of 30 tests run: 29 passed, 1 failed. 96.7% <- summary for `props': of 30 tests run: 30 passed, 0 failed. 100.0% Change-Id: Ibe51f0199afaab6b95894aed2244f1c4becad3da Reviewed-on: https://go-review.googlesource.com/10304Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 20 May, 2015 1 commit
-
-
Nigel Tao authored
This makes properties consistent even after a COPY, MOVE or DELETE. A follow-up CL will eliminate the concept of a configurable PropSystem entirely. See discussion at https://groups.google.com/forum/#!topic/golang-dev/2_LiN6sf93A Litmust test before/after: <- summary for `props': of 30 tests run: 28 passed, 2 failed. 93.3% <- summary for `props': of 30 tests run: 29 passed, 1 failed. 96.7% Change-Id: I9bd09d9fb73e40f87306eaec2679945874af023d Reviewed-on: https://go-review.googlesource.com/10241Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 19 May, 2015 1 commit
-
-
Nigel Tao authored
Package webdav requires the Go standard library's encoding/xml package version 1.5 or greater. Fixes #10904 Change-Id: Idf2915e581f4efa6f00e7701785ea258f9e1a8f4 Reviewed-on: https://go-review.googlesource.com/10243Reviewed-by: Robert Stepanek <robert.stepanek@gmail.com> Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 18 May, 2015 1 commit
-
-
Dave Cheney authored
Fixes #10866 When ws.Close is called, always close the underlying ws.rwc. Change-Id: Ia709d5e0bc51ffb7194668d2764848d012e0c652 Reviewed-on: https://go-review.googlesource.com/10135Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 15 May, 2015 2 commits
-
-
Mikio Hara authored
Change-Id: I519fbce7681b18694336482e9135731647ed3efa Reviewed-on: https://go-review.googlesource.com/10095Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Mikio Hara authored
Change-Id: I16e3edd48a37ec385c24bd6de256ea896c8df652 Reviewed-on: https://go-review.googlesource.com/10094Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 08 May, 2015 1 commit
-
-
Ian Lance Taylor authored
Update the licensing info following the instructions at http://www.w3.org/Consortium/Legal/2008/04-testsuite-copyright.html#howtouse Fixes golang/go#10398. Change-Id: Ib37b2a696a5287f41d4e85da4eb7ec1cbb346301 Reviewed-on: https://go-review.googlesource.com/8978Reviewed-by: Martin Packman <martin.packman@canonical.com> Reviewed-by: Rob Pike <r@golang.org>
-
- 29 Apr, 2015 1 commit
-
-
Robert Stepanek authored
This CL adds support to store arbitrary WebDAV properties in the in-memory property system reference implementation. It covers the the majority of property related litmus test cases. However, this CL does not add support for COPY/MOVE/DELETE requests to the PropSystem interface or implementation; memory occupied by dead properties of affected resources will not be released. I propose to first agree on how to store and lock dead properties in this CL and add support for COPY/MOVE/DELETE in a follow-up change. Before: Coverage of litmus 'props' test suite 16 tests were skipped, 14 tests run. 10 passed, 4 failed. 71.4% After: Coverage of litmus 'props' test suite 0 tests were skipped, 30 tests run. 28 passed, 2 failed. 93.3% Change-Id: Ie9af665fc588332ed30c7de256f47f8405078db3 Reviewed-on: https://go-review.googlesource.com/9374Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 24 Apr, 2015 1 commit
-
-
Brad Fitzpatrick authored
The SPDY package has known bugs (9 currently) and incompleteness. Rather than fix and finish it, delete it. Browsers are removing SPDY support anyway. The future is HTTP/2. Anybody who wants the old x/net/spdy can get it from git revision d175081d and vendor it. Fixes golang/go#10547 Fixes golang/go#10545 Fixes golang/go#10544 Fixes golang/go#10543 Fixes golang/go#10542 Fixes golang/go#10540 Fixes golang/go#10539 Fixes golang/go#6037 Fixes golang/go#4295 Change-Id: I98ede7f8da00af64015d0f2784cadba33286a6a5 Reviewed-on: https://go-review.googlesource.com/9300Reviewed-by: Dave Cheney <dave@cheney.net> Reviewed-by: Rob Pike <r@golang.org> Reviewed-by: David Symonds <dsymonds@golang.org> Reviewed-by: Nigel Tao <nigeltao@golang.org> Reviewed-by: Dave Day <djd@golang.org>
-
- 23 Apr, 2015 1 commit
-
-
Robert Stepanek authored
Currently, the package-internal multistatus writer does not allow to write empty XML multistatus elements. This conflicts with RFC 4918 (see [1]), and prevents to reuse the writer to implement REPORTs and other WebDAV extensions. This CL adds support to write empty multi- status responses. Based on this, it simplifies the error handling of PROPFIND and PROPPATCH request handling. [1] http://webdav.org/specs/rfc4918.html#ELEMENT_multistatus Change-Id: I6ae56ec261ad3a0763b6c197ad1e9c2e30d988a6 Reviewed-on: https://go-review.googlesource.com/9186Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 22 Apr, 2015 1 commit
-
-
Alex Brainman authored
walkFS is used in TestWalkFS to walk over memFS FileSystem. memFS uses slash delimited paths, so we must use path, not filepath, package to construct path elements in walkFS. Fixes golang/go#10517. Change-Id: I3632472e5fb0fbaa13e5656d9ae5483e8ec332b2 Reviewed-on: https://go-review.googlesource.com/9193Reviewed-by: Robert Stepanek <robert.stepanek@gmail.com> Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 20 Apr, 2015 1 commit
-
-
Robert Stepanek authored
This change adds support to parse and handle PROPPATCH requests. It adds the Patch method to the PropSystem interface. In order to keep this CL small, it does not add support to set "dead" DAV properties. Instead, any patched property is reported with HTTP status code 403 Forbidden. Once this CL is accepted, I will submit a CL to store dead DAV properties in-memory. The litmus test coverage of the 'props' test suite remains as-is: 16 tests were skipped, 14 tests run. 10 passed, 4 failed. 71.4% Change-Id: I14a25464e94b3316c16976f79b4457bf646d0d24 Reviewed-on: https://go-review.googlesource.com/8937Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 09 Apr, 2015 1 commit
-
-
Robert Stepanek authored
This change adds ETag headers to GET/HEAD/POST and PUT responses. It does not update the existing If-header request handling. The ETag header and DAV property value can be overriden by implementing a custom property system. A similar scheme is provided for Content-Type. This CL makes net/webdav pass three more litmus ‘locks’ test cases successfully. Before: Summary for `locks': of 30 tests run: 27 passed, 3 failed. 90.0% After: Summary for `locks': of 34 tests run: 30 passed, 4 failed. 88.2% Change-Id: I5102b9ac18d20844deaaa630b62cc3611b3f0740 Reviewed-on: https://go-review.googlesource.com/4903Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 03 Apr, 2015 1 commit
-
-
Andy Balholm authored
Change-Id: Ic4d1df0c4f7048a3e2472cca09ef9390bcfd149d Reviewed-on: https://go-review.googlesource.com/4533Reviewed-by: Rob Pike <r@golang.org>
-
- 20 Mar, 2015 3 commits
-
-
Robert Stepanek authored
This change adds support for PROPFIND requests to net/webdav. It contains a proposed PropSystem interface and a preliminary implementation of an in-memory property system. As discussed with nigeltao, this is the first of approximately 4-5 CLs to get property support in the net/webdav package. Current coverage of litmus 'props' test suite: 16 tests were skipped, 14 tests run. 10 passed, 4 failed. 71.4% Change-Id: I0bc5f375422137e911a2f6fb0e99c43a5a52d5ac Reviewed-on: https://go-review.googlesource.com/3417Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
Mikio Hara authored
Change-Id: Ieba3be87526fab5fbc545735e0b721bfd5aea657 Reviewed-on: https://go-review.googlesource.com/7650Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Mikio Hara authored
Change-Id: I30996019437892e09bfe8710c903a510c96f5d13 Reviewed-on: https://go-review.googlesource.com/7619Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 18 Mar, 2015 1 commit
-
-
Josh Bleecher Snyder authored
See golang.org/cl/4131 for context. Change-Id: Iec4aa97042381e0ebb62f1361f3c71e3dee1c569 Reviewed-on: https://go-review.googlesource.com/7727Reviewed-by: David Crawshaw <crawshaw@golang.org>
-
- 15 Mar, 2015 2 commits
-
-
Mikio Hara authored
Fixes golang/go#10164. Change-Id: Ifcef660e4840cfcfa2f72a2be08541cad2b90440 Reviewed-on: https://go-review.googlesource.com/7611Reviewed-by: Minux Ma <minux@golang.org>
-
Mikio Hara authored
Fixes golang/go#10164. Change-Id: I68b50ee4d516b3c9b6898e05b37a240f9592e19a Reviewed-on: https://go-review.googlesource.com/7610Reviewed-by: Minux Ma <minux@golang.org>
-
- 14 Mar, 2015 3 commits
-
-
Mikio Hara authored
Change-Id: I392835ac2ceddbdfc9f23e5a66abd16f7fd88b70 Reviewed-on: https://go-review.googlesource.com/6803Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Mikio Hara authored
Change-Id: I8a387f0fca0985c1ff059052173adee44dcdce90 Reviewed-on: https://go-review.googlesource.com/6804Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Mikio Hara authored
Change-Id: Ie5712a49f0c909538f4c36311b591e930a77efb5 Reviewed-on: https://go-review.googlesource.com/5410Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 04 Mar, 2015 1 commit
-
-
Robert Stepanek authored
The change to the standard encoding/xml library was https://go-review.googlesource.com/#/c/2660/ which landed on 2015-02-14. An additional change was https://go-review.googlesource.com/#/c/5910/ which landed on 2015-03-03. Fixes #9978. Change-Id: I4f798f153e4e13b13eadc10e44b21a4b118251d3 Reviewed-on: https://go-review.googlesource.com/5714Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 03 Mar, 2015 1 commit
-
-
Dmitry Savintsev authored
The current documentation as well as set of atoms and attributes has gotten slightly out of sync with the current state of the WHATWG html5 specification. The change adds and removes several of the atoms and attributes, updates the documentation (such as steps numbering in inBodyEndTagFormatting) and modifies the spec URLs to https:// Change-Id: I6dfa52785858c1521301b20b1e585e19a08b1e98 Reviewed-on: https://go-review.googlesource.com/6173Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
- 20 Feb, 2015 1 commit
-
-
Aaron Jacobs authored
This is necessary for cleanup of any goroutine that blocks on ctx.Done(), as is done in e.g. propagateCancel in context.go. It is already specified for WithTimeout and WithDeadline, whose descriptions I have simplified here. Change-Id: I6b30605decb8c0b7c1296c3f8852f9ad305ad4b7 Reviewed-on: https://go-review.googlesource.com/5022Reviewed-by: Sameer Ajmani <sameer@golang.org>
-
- 18 Feb, 2015 1 commit
-
-
Andrew Gerrand authored
Change-Id: Ie44649d1d5594ba140e673eaafb94fecead561ee Reviewed-on: https://go-review.googlesource.com/5214Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
- 12 Feb, 2015 1 commit
-
-
Bryan C. Mills authored
The current example leaks the DoSomethingSlow goroutine for an arbitrarily long time. In a real server, that can lead to an out-of-memory failure during events such as network outages; a more careful version of that example would be too long for a simple package doc. Fortunately, there are other short, common patterns using Done that don't leak and don't require a lot of explanation. Let's use one of those instead. Change-Id: I0ad0c6121d06b757a397e0e71be9e01ccfd75f77 Reviewed-on: https://go-review.googlesource.com/4490Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 10 Feb, 2015 2 commits
-
-
Andy Balholm authored
This provides a CharsetReader function for xml.Decoder. Change-Id: Id00787bbdee90d267d38c84c98a06f9e10d93336 Reviewed-on: https://go-review.googlesource.com/4420Reviewed-by: Nigel Tao <nigeltao@golang.org>
-
Nigel Tao authored
LockSystem.Confirm now takes two names (a src and a dst), and returns a func() instead of a 1-method Releaser. We now pass 25 of 30 "locks" litmus tests: - 4 failures are due to PROPFIND / PROPPATCH not being implemented. - 1 failure is due to shared locks being unsupported. - 3 warnings are also due to PROPFIND / PROPPATCH. - 1 failure (cond_put_corrupt_token) is due to returning 412 (Precondition Failed) instead of 423 (Locked), but IIUC the spec says to return 412. - 11 tests were skipped, presumably due to earlier failures. Change-Id: I3f4c178cdc4b99c6acb7f59783b4fd9b94f606ec Reviewed-on: https://go-review.googlesource.com/3860Reviewed-by: Dave Cheney <dave@cheney.net>
-