From f1d56f24371c3c234fa51a9a78fa3bc7d9c7dc11 Mon Sep 17 00:00:00 2001
From: Simon Ser <contact@emersion.fr>
Date: Wed, 7 Feb 2024 17:22:41 +0100
Subject: [PATCH] internal: add IsRequestEmpty

---
 carddav/server.go  | 10 ++--------
 internal/server.go |  5 +++++
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/carddav/server.go b/carddav/server.go
index 52e3523..5e0b259 100644
--- a/carddav/server.go
+++ b/carddav/server.go
@@ -5,7 +5,6 @@ import (
 	"context"
 	"encoding/xml"
 	"fmt"
-	"io"
 	"mime"
 	"net/http"
 	"path"
@@ -703,13 +702,7 @@ func (b *backend) Mkcol(r *http.Request) error {
 		Path: r.URL.Path,
 	}
 
-	// Check if a request body was sent
-	_, err := r.Body.Read(nil)
-	if err != nil && err != io.EOF {
-		return err
-	}
-	if err == nil {
-		// Not EOF, body is present
+	if !internal.IsRequestBodyEmpty(r) {
 		var m mkcolReq
 		if err := internal.DecodeXMLRequest(r, &m); err != nil {
 			return internal.HTTPErrorf(http.StatusBadRequest, "carddav: error parsing mkcol request: %s", err.Error())
@@ -722,6 +715,7 @@ func (b *backend) Mkcol(r *http.Request) error {
 		ab.Description = m.Description.Description
 		// TODO ...
 	}
+
 	return b.Backend.CreateAddressBook(r.Context(), ab)
 }
 
diff --git a/internal/server.go b/internal/server.go
index f9c7b68..85ed1bc 100644
--- a/internal/server.go
+++ b/internal/server.go
@@ -44,6 +44,11 @@ func DecodeXMLRequest(r *http.Request, v interface{}) error {
 	return nil
 }
 
+func IsRequestBodyEmpty(r *http.Request) bool {
+	_, err := r.Body.Read(nil)
+	return err == io.EOF
+}
+
 func ServeXML(w http.ResponseWriter) *xml.Encoder {
 	w.Header().Add("Content-Type", "text/xml; charset=\"utf-8\"")
 	w.Write([]byte(xml.Header))