fix(calendar): GET auf CalDAV-Sammlungen antwortet statt 404
Manche Clients folgen dem 308-Redirect von /.well-known/caldav mit GET statt PROPFIND (curl -L reproduziert das exakt), landen dann auf einer Sammlung (Principal/Home-Set/Kalender) und bekamen dort bisher ein hartes 404 von GetCalendarObject – ein Client, der so die Erreichbarkeit prüft, bricht die Einrichtung dann vorzeitig ab, bevor er je ein PROPFIND schickt. GET/HEAD auf Nicht-.ics-Pfade antworten jetzt mit einer schlichten 200-Textantwort statt an den CalDAV-Backend durchgereicht zu werden. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
1 parent
d6e1854f1b
commit
469274171c
1 file changed
+17
@@ -1,8 +1,10 @@
|
|||||||
package calendar
|
package calendar
|
||||||
|
|
||||||
import (
|
import (
|
||||||
|
"io"
|
||||||
"net/http"
|
"net/http"
|
||||||
"strconv"
|
"strconv"
|
||||||
|
"strings"
|
||||||
"time"
|
"time"
|
||||||
|
|
||||||
"github.com/emersion/go-webdav/caldav"
|
"github.com/emersion/go-webdav/caldav"
|
||||||
@@ -58,6 +60,21 @@ func (h *Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
|
|||||||
}
|
}
|
||||||
h.limiter.Reset(clientIP)
|
h.limiter.Reset(clientIP)
|
||||||
|
|
||||||
|
// Manche Clients folgen dem 308-Redirect von /.well-known/caldav mit
|
||||||
|
// GET statt PROPFIND, oder ein Mensch öffnet die Adresse zum Testen im
|
||||||
|
// Browser. Ohne diese Abkürzung würde das an GetCalendarObject
|
||||||
|
// durchgereicht, das für Sammlungen (Principal, Home-Set, Kalender) mit
|
||||||
|
// 404 antwortet – ein Client, der so den Dienst prüft, bräche die
|
||||||
|
// Einrichtung ab, bevor er je ein PROPFIND schickt.
|
||||||
|
if (r.Method == http.MethodGet || r.Method == http.MethodHead) && !strings.HasSuffix(r.URL.Path, ".ics") {
|
||||||
|
w.Header().Set("Content-Type", "text/plain; charset=utf-8")
|
||||||
|
w.WriteHeader(http.StatusOK)
|
||||||
|
if r.Method == http.MethodGet {
|
||||||
|
io.WriteString(w, "Dies ist eine CalDAV-Sammlung. Mit einem CalDAV-Client verbinden (PROPFIND), nicht mit dem Browser.\n")
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
h.dav.ServeHTTP(w, r.WithContext(contextWithUser(r.Context(), user)))
|
h.dav.ServeHTTP(w, r.WithContext(contextWithUser(r.Context(), user)))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in new issue
Block a user