1f349/go-webdav
4
0
Fork 0
mirror of https://github.com/1f349/go-webdav.git synced 2024-12-22 08:14:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

carddav/caldav: use 308 for .well-known redirects

Browse Source 
This makes it a little less ambiguous (and, in case of Go clients, a lot
easier) that clients should follow the redirect by sending the same
PROPFIND request, including body, to the new location.

See also the documentation of [http.Client.Do()][1] and the comments in
[http.redirectBehavior()][2].

Confirmed to not make a difference for Evolution and Thunderbird
clients.

[1]: https://pkg.go.dev/net/http#Client.Do
[2]: https://cs.opensource.google/go/go/+/refs/tags/go1.18.2:src/net/http/client.go;drc=d8762b2f4532cc2e5ec539670b88bbc469a13938;l=502
This commit is contained in:
Conrad Hoffmann 2022-05-20 15:58:52 +02:00 committed by Simon Ser
parent bc3faca3a0
commit b0c59cdea1
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
caldav/server.go
View File

@ -58,7 +58,7 @@ func (h *Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
return return
} }
http.Redirect(w, r, principalPath, http.StatusMovedPermanently) http.Redirect(w, r, principalPath, http.StatusPermanentRedirect)
return return
} }

2
carddav/server.go
View File

@ -57,7 +57,7 @@ func (h *Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
return return
} }
http.Redirect(w, r, principalPath, http.StatusMovedPermanently) http.Redirect(w, r, principalPath, http.StatusPermanentRedirect)
return return
} }