Add error returning for missing error output paths

Co-Authored-By: Captain ALM <captainalm@captainalm.com>

caldav/server.go

@@ -465,10 +465,12 @@ func (b *backend) propFindRoot(ctx context.Context, propfind *internal.PropFind)
 	}
 
 	props := map[xml.Name]internal.PropFindFunc{
-		internal.CurrentUserPrincipalName: internal.PropFindValue(&internal.CurrentUserPrincipal{
+		internal.CurrentUserPrincipalName: func(*internal.RawXMLValue) (interface{}, error) {
-			Href: internal.Href{Path: principalPath},
+			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: principalPath}}, nil
-		}),
+		},
-		internal.ResourceTypeName: internal.PropFindValue(internal.NewResourceType(internal.CollectionName)),
+		internal.ResourceTypeName: func(*internal.RawXMLValue) (interface{}, error) {
+			return internal.NewResourceType(internal.CollectionName), nil
+		},
 	}
 	return internal.NewPropFindResponse(principalPath, propfind, props)
 }
@@ -484,13 +486,15 @@ func (b *backend) propFindUserPrincipal(ctx context.Context, propfind *internal.
 	}
 
 	props := map[xml.Name]internal.PropFindFunc{
-		internal.CurrentUserPrincipalName: internal.PropFindValue(&internal.CurrentUserPrincipal{
+		internal.CurrentUserPrincipalName: func(*internal.RawXMLValue) (interface{}, error) {
-			Href: internal.Href{Path: principalPath},
+			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: principalPath}}, nil
-		}),
+		},
-		calendarHomeSetName: internal.PropFindValue(&calendarHomeSet{
+		calendarHomeSetName: func(*internal.RawXMLValue) (interface{}, error) {
-			Href: internal.Href{Path: homeSetPath},
+			return &calendarHomeSet{Href: internal.Href{Path: homeSetPath}}, nil
-		}),
+		},
-		internal.ResourceTypeName: internal.PropFindValue(internal.NewResourceType(internal.CollectionName)),
+		internal.ResourceTypeName: func(*internal.RawXMLValue) (interface{}, error) {
+			return internal.NewResourceType(internal.CollectionName), nil
+		},
 	}
 	return internal.NewPropFindResponse(principalPath, propfind, props)
 }
@@ -507,10 +511,12 @@ func (b *backend) propFindHomeSet(ctx context.Context, propfind *internal.PropFi
 
 	// TODO anything else to return here?
 	props := map[xml.Name]internal.PropFindFunc{
-		internal.CurrentUserPrincipalName: internal.PropFindValue(&internal.CurrentUserPrincipal{
+		internal.CurrentUserPrincipalName: func(*internal.RawXMLValue) (interface{}, error) {
-			Href: internal.Href{Path: principalPath},
+			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: principalPath}}, nil
-		}),
+		},
-		internal.ResourceTypeName: internal.PropFindValue(internal.NewResourceType(internal.CollectionName)),
+		internal.ResourceTypeName: func(*internal.RawXMLValue) (interface{}, error) {
+			return internal.NewResourceType(internal.CollectionName), nil
+		},
 	}
 	return internal.NewPropFindResponse(homeSetPath, propfind, props)
 }
@@ -524,15 +530,19 @@ func (b *backend) propFindCalendar(ctx context.Context, propfind *internal.PropF
 			}
 			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: path}}, nil
 		},
-		internal.ResourceTypeName: internal.PropFindValue(internal.NewResourceType(internal.CollectionName, calendarName)),
+		internal.ResourceTypeName: func(*internal.RawXMLValue) (interface{}, error) {
-		calendarDescriptionName: internal.PropFindValue(&calendarDescription{
+			return internal.NewResourceType(internal.CollectionName, calendarName), nil
-			Description: cal.Description,
+		},
-		}),
+		calendarDescriptionName: func(*internal.RawXMLValue) (interface{}, error) {
-		supportedCalendarDataName: internal.PropFindValue(&supportedCalendarData{
+			return &calendarDescription{Description: cal.Description}, nil
-			Types: []calendarDataType{
+		},
-				{ContentType: ical.MIMEType, Version: "2.0"},
+		supportedCalendarDataName: func(*internal.RawXMLValue) (interface{}, error) {
-			},
+			return &supportedCalendarData{
-		}),
+				Types: []calendarDataType{
+					{ContentType: ical.MIMEType, Version: "2.0"},
+				},
+			}, nil
+		},
 		supportedCalendarComponentSetName: func(*internal.RawXMLValue) (interface{}, error) {
 			components := []comp{}
 			if cal.SupportedComponentSet != nil {
@@ -549,19 +559,19 @@ func (b *backend) propFindCalendar(ctx context.Context, propfind *internal.PropF
 	}
 
 	if cal.Name != "" {
-		props[internal.DisplayNameName] = internal.PropFindValue(&internal.DisplayName{
+		props[internal.DisplayNameName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Name: cal.Name,
+			return &internal.DisplayName{Name: cal.Name}, nil
-		})
+		}
 	}
 	if cal.Description != "" {
-		props[calendarDescriptionName] = internal.PropFindValue(&calendarDescription{
+		props[calendarDescriptionName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Description: cal.Description,
+			return &calendarDescription{Description: cal.Description}, nil
-		})
+		}
 	}
 	if cal.MaxResourceSize > 0 {
-		props[maxResourceSizeName] = internal.PropFindValue(&maxResourceSize{
+		props[maxResourceSizeName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Size: cal.MaxResourceSize,
+			return &maxResourceSize{Size: cal.MaxResourceSize}, nil
-		})
+		}
 	}
 	props[internal.CurrentUserPrivilegeSetName] = func(*internal.RawXMLValue) (interface{}, error) {
 		return &internal.CurrentUserPrivilegeSet{Privilege: internal.NewAllPrivileges()}, nil
@@ -605,9 +615,9 @@ func (b *backend) propFindCalendarObject(ctx context.Context, propfind *internal
 			}
 			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: path}}, nil
 		},
-		internal.GetContentTypeName: internal.PropFindValue(&internal.GetContentType{
+		internal.GetContentTypeName: func(*internal.RawXMLValue) (interface{}, error) {
-			Type: ical.MIMEType,
+			return &internal.GetContentType{Type: ical.MIMEType}, nil
-		}),
+		},
 		// TODO: calendar-data can only be used in REPORT requests
 		calendarDataName: func(*internal.RawXMLValue) (interface{}, error) {
 			var buf bytes.Buffer
@@ -620,20 +630,20 @@ func (b *backend) propFindCalendarObject(ctx context.Context, propfind *internal
 	}
 
 	if co.ContentLength > 0 {
-		props[internal.GetContentLengthName] = internal.PropFindValue(&internal.GetContentLength{
+		props[internal.GetContentLengthName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Length: co.ContentLength,
+			return &internal.GetContentLength{Length: co.ContentLength}, nil
-		})
+		}
 	}
 	if !co.ModTime.IsZero() {
-		props[internal.GetLastModifiedName] = internal.PropFindValue(&internal.GetLastModified{
+		props[internal.GetLastModifiedName] = func(*internal.RawXMLValue) (interface{}, error) {
-			LastModified: internal.Time(co.ModTime),
+			return &internal.GetLastModified{LastModified: internal.Time(co.ModTime)}, nil
-		})
+		}
 	}
 
 	if co.ETag != "" {
-		props[internal.GetETagName] = internal.PropFindValue(&internal.GetETag{
+		props[internal.GetETagName] = func(*internal.RawXMLValue) (interface{}, error) {
-			ETag: internal.ETag(co.ETag),
+			return &internal.GetETag{ETag: internal.ETag(co.ETag)}, nil
-		})
+		}
 	}
 
 	return internal.NewPropFindResponse(co.Path, propfind, props)

carddav/server.go

@@ -431,10 +431,12 @@ func (b *backend) propFindRoot(ctx context.Context, propfind *internal.PropFind)
 	}
 
 	props := map[xml.Name]internal.PropFindFunc{
-		internal.CurrentUserPrincipalName: internal.PropFindValue(&internal.CurrentUserPrincipal{
+		internal.CurrentUserPrincipalName: func(*internal.RawXMLValue) (interface{}, error) {
-			Href: internal.Href{Path: principalPath},
+			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: principalPath}}, nil
-		}),
+		},
-		internal.ResourceTypeName: internal.PropFindValue(internal.NewResourceType(internal.CollectionName)),
+		internal.ResourceTypeName: func(*internal.RawXMLValue) (interface{}, error) {
+			return internal.NewResourceType(internal.CollectionName), nil
+		},
 	}
 	return internal.NewPropFindResponse(principalPath, propfind, props)
 }
@@ -444,24 +446,30 @@ func (b *backend) propFindUserPrincipal(ctx context.Context, propfind *internal.
 	if err != nil {
 		return nil, err
 	}
+	homeSetPath, err := b.Backend.AddressBookHomeSetPath(ctx)
+	if err != nil {
+		return nil, err
+	}
 
 	props := map[xml.Name]internal.PropFindFunc{
-		internal.CurrentUserPrincipalName: internal.PropFindValue(&internal.CurrentUserPrincipal{
+		internal.CurrentUserPrincipalName: func(*internal.RawXMLValue) (interface{}, error) {
-			Href: internal.Href{Path: principalPath},
+			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: principalPath}}, nil
-		}),
+		},
 		addressBookHomeSetName: func(*internal.RawXMLValue) (interface{}, error) {
-			homeSetPath, err := b.Backend.AddressBookHomeSetPath(ctx)
-			if err != nil {
-				return nil, err
-			}
 			return &addressbookHomeSet{Href: internal.Href{Path: homeSetPath}}, nil
 		},
-		internal.ResourceTypeName: internal.PropFindValue(internal.NewResourceType(internal.CollectionName)),
+		internal.ResourceTypeName: func(*internal.RawXMLValue) (interface{}, error) {
+			return internal.NewResourceType(internal.CollectionName), nil
+		},
 	}
 	return internal.NewPropFindResponse(principalPath, propfind, props)
 }
 
 func (b *backend) propFindHomeSet(ctx context.Context, propfind *internal.PropFind) (*internal.Response, error) {
+	principalPath, err := b.Backend.CurrentUserPrincipal(ctx)
+	if err != nil {
+		return nil, err
+	}
 	homeSetPath, err := b.Backend.AddressBookHomeSetPath(ctx)
 	if err != nil {
 		return nil, err
@@ -470,13 +478,11 @@ func (b *backend) propFindHomeSet(ctx context.Context, propfind *internal.PropFi
 	// TODO anything else to return here?
 	props := map[xml.Name]internal.PropFindFunc{
 		internal.CurrentUserPrincipalName: func(*internal.RawXMLValue) (interface{}, error) {
-			principalPath, err := b.Backend.CurrentUserPrincipal(ctx)
-			if err != nil {
-				return nil, err
-			}
 			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: principalPath}}, nil
 		},
-		internal.ResourceTypeName: internal.PropFindValue(internal.NewResourceType(internal.CollectionName)),
+		internal.ResourceTypeName: func(*internal.RawXMLValue) (interface{}, error) {
+			return internal.NewResourceType(internal.CollectionName), nil
+		},
 	}
 	return internal.NewPropFindResponse(homeSetPath, propfind, props)
 }
@@ -490,29 +496,33 @@ func (b *backend) propFindAddressBook(ctx context.Context, propfind *internal.Pr
 			}
 			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: path}}, nil
 		},
-		internal.ResourceTypeName: internal.PropFindValue(internal.NewResourceType(internal.CollectionName, addressBookName)),
+		internal.ResourceTypeName: func(*internal.RawXMLValue) (interface{}, error) {
-		supportedAddressDataName: internal.PropFindValue(&supportedAddressData{
+			return internal.NewResourceType(internal.CollectionName, addressBookName), nil
-			Types: []addressDataType{
+		},
-				{ContentType: vcard.MIMEType, Version: "3.0"},
+		supportedAddressDataName: func(*internal.RawXMLValue) (interface{}, error) {
-				{ContentType: vcard.MIMEType, Version: "4.0"},
+			return &supportedAddressData{
-			},
+				Types: []addressDataType{
-		}),
+					{ContentType: vcard.MIMEType, Version: "3.0"},
+					{ContentType: vcard.MIMEType, Version: "4.0"},
+				},
+			}, nil
+		},
 	}
 
 	if ab.Name != "" {
-		props[internal.DisplayNameName] = internal.PropFindValue(&internal.DisplayName{
+		props[internal.DisplayNameName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Name: ab.Name,
+			return &internal.DisplayName{Name: ab.Name}, nil
-		})
+		}
 	}
 	if ab.Description != "" {
-		props[addressBookDescriptionName] = internal.PropFindValue(&addressbookDescription{
+		props[addressBookDescriptionName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Description: ab.Description,
+			return &addressbookDescription{Description: ab.Description}, nil
-		})
+		}
 	}
 	if ab.MaxResourceSize > 0 {
-		props[maxResourceSizeName] = internal.PropFindValue(&maxResourceSize{
+		props[maxResourceSizeName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Size: ab.MaxResourceSize,
+			return &maxResourceSize{Size: ab.MaxResourceSize}, nil
-		})
+		}
 	}
 	props[internal.CurrentUserPrivilegeSetName] = func(*internal.RawXMLValue) (interface{}, error) {
 		return &internal.CurrentUserPrivilegeSet{Privilege: internal.NewAllPrivileges()}, nil
@@ -554,9 +564,9 @@ func (b *backend) propFindAddressObject(ctx context.Context, propfind *internal.
 			}
 			return &internal.CurrentUserPrincipal{Href: internal.Href{Path: path}}, nil
 		},
-		internal.GetContentTypeName: internal.PropFindValue(&internal.GetContentType{
+		internal.GetContentTypeName: func(*internal.RawXMLValue) (interface{}, error) {
-			Type: vcard.MIMEType,
+			return &internal.GetContentType{Type: vcard.MIMEType}, nil
-		}),
+		},
 		// TODO: address-data can only be used in REPORT requests
 		addressDataName: func(*internal.RawXMLValue) (interface{}, error) {
 			var buf bytes.Buffer
@@ -569,20 +579,20 @@ func (b *backend) propFindAddressObject(ctx context.Context, propfind *internal.
 	}
 
 	if ao.ContentLength > 0 {
-		props[internal.GetContentLengthName] = internal.PropFindValue(&internal.GetContentLength{
+		props[internal.GetContentLengthName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Length: ao.ContentLength,
+			return &internal.GetContentLength{Length: ao.ContentLength}, nil
-		})
+		}
 	}
 	if !ao.ModTime.IsZero() {
-		props[internal.GetLastModifiedName] = internal.PropFindValue(&internal.GetLastModified{
+		props[internal.GetLastModifiedName] = func(*internal.RawXMLValue) (interface{}, error) {
-			LastModified: internal.Time(ao.ModTime),
+			return &internal.GetLastModified{LastModified: internal.Time(ao.ModTime)}, nil
-		})
+		}
 	}
 
 	if ao.ETag != "" {
-		props[internal.GetETagName] = internal.PropFindValue(&internal.GetETag{
+		props[internal.GetETagName] = func(*internal.RawXMLValue) (interface{}, error) {
-			ETag: internal.ETag(ao.ETag),
+			return &internal.GetETag{ETag: internal.ETag(ao.ETag)}, nil
-		})
+		}
 	}
 
 	return internal.NewPropFindResponse(ao.Path, propfind, props)

fs_local.go

@@ -114,31 +114,15 @@ func (fs LocalFileSystem) ReadDir(ctx context.Context, name string, recursive bo
 	return l, errFromOS(err)
 }
 
-func (fs LocalFileSystem) Create(ctx context.Context, name string, body io.ReadCloser, opts *CreateOptions) (fi *FileInfo, created bool, err error) {
+func (fs LocalFileSystem) Create(ctx context.Context, name string, body io.ReadCloser) (*FileInfo, bool, error) {
 	p, err := fs.localPath(name)
 	if err != nil {
 		return nil, false, err
 	}
-	fi, _ = fs.Stat(ctx, name)
+	created := false
-	created = fi == nil
+	fi, _ := fs.Stat(ctx, name)
-	etag := ""
+	if fi == nil {
-	if fi != nil {
+		created = true
-		etag = fi.ETag
-	}
-
-	if opts.IfMatch.IsSet() {
-		if ok, err := opts.IfMatch.MatchETag(etag); err != nil {
-			return nil, false, NewHTTPError(http.StatusBadRequest, err)
-		} else if !ok {
-			return nil, false, NewHTTPError(http.StatusPreconditionFailed, fmt.Errorf("If-Match condition failed"))
-		}
-	}
-	if opts.IfNoneMatch.IsSet() {
-		if ok, err := opts.IfNoneMatch.MatchETag(etag); err != nil {
-			return nil, false, NewHTTPError(http.StatusBadRequest, err)
-		} else if ok {
-			return nil, false, NewHTTPError(http.StatusPreconditionFailed, fmt.Errorf("If-None-Match condition failed"))
-		}
 	}
 
 	wc, err := os.Create(p)
@@ -148,11 +132,9 @@ func (fs LocalFileSystem) Create(ctx context.Context, name string, body io.ReadC
 	defer wc.Close()
 
 	if _, err := io.Copy(wc, body); err != nil {
-		os.Remove(p)
 		return nil, false, err
 	}
 	if err := wc.Close(); err != nil {
-		os.Remove(p)
 		return nil, false, err
 	}
 

internal/server.go

@@ -162,17 +162,13 @@ func (h *Handler) handlePropfind(w http.ResponseWriter, r *http.Request) error {
 
 type PropFindFunc func(raw *RawXMLValue) (interface{}, error)
 
-func PropFindValue(value interface{}) PropFindFunc {
-	return func(raw *RawXMLValue) (interface{}, error) {
-		return value, nil
-	}
-}
-
 func NewPropFindResponse(path string, propfind *PropFind, props map[xml.Name]PropFindFunc) (*Response, error) {
 	resp := &Response{Hrefs: []Href{Href{Path: path}}}
 
 	if _, ok := props[ResourceTypeName]; !ok {
-		props[ResourceTypeName] = PropFindValue(NewResourceType())
+		props[ResourceTypeName] = func(*RawXMLValue) (interface{}, error) {
+			return NewResourceType(), nil
+		}
 	}
 
 	if propfind.PropName != nil {

server.go

@@ -17,7 +17,7 @@ type FileSystem interface {
 	Open(ctx context.Context, name string) (io.ReadCloser, error)
 	Stat(ctx context.Context, name string) (*FileInfo, error)
 	ReadDir(ctx context.Context, name string, recursive bool) ([]FileInfo, error)
-	Create(ctx context.Context, name string, body io.ReadCloser, opts *CreateOptions) (fileInfo *FileInfo, created bool, err error)
+	Create(ctx context.Context, name string, body io.ReadCloser) (fileInfo *FileInfo, created bool, err error)
 	RemoveAll(ctx context.Context, name string) error
 	Mkdir(ctx context.Context, name string) error
 	Copy(ctx context.Context, name, dest string, options *CopyOptions) (created bool, err error)
@@ -162,26 +162,26 @@ func (b *backend) propFindFile(propfind *internal.PropFind, fi *FileInfo) (*inte
 	}
 
 	if !fi.IsDir {
-		props[internal.GetContentLengthName] = internal.PropFindValue(&internal.GetContentLength{
+		props[internal.GetContentLengthName] = func(*internal.RawXMLValue) (interface{}, error) {
-			Length: fi.Size,
+			return &internal.GetContentLength{Length: fi.Size}, nil
-		})
+		}
 
 		if !fi.ModTime.IsZero() {
-			props[internal.GetLastModifiedName] = internal.PropFindValue(&internal.GetLastModified{
+			props[internal.GetLastModifiedName] = func(*internal.RawXMLValue) (interface{}, error) {
-				LastModified: internal.Time(fi.ModTime),
+				return &internal.GetLastModified{LastModified: internal.Time(fi.ModTime)}, nil
-			})
+			}
 		}
 
 		if fi.MIMEType != "" {
-			props[internal.GetContentTypeName] = internal.PropFindValue(&internal.GetContentType{
+			props[internal.GetContentTypeName] = func(*internal.RawXMLValue) (interface{}, error) {
-				Type: fi.MIMEType,
+				return &internal.GetContentType{Type: fi.MIMEType}, nil
-			})
+			}
 		}
 
 		if fi.ETag != "" {
-			props[internal.GetETagName] = internal.PropFindValue(&internal.GetETag{
+			props[internal.GetETagName] = func(*internal.RawXMLValue) (interface{}, error) {
-				ETag: internal.ETag(fi.ETag),
+				return &internal.GetETag{ETag: internal.ETag(fi.ETag)}, nil
-			})
+			}
 		}
 	}
 
@@ -194,14 +194,7 @@ func (b *backend) PropPatch(r *http.Request, update *internal.PropertyUpdate) (*
 }
 
 func (b *backend) Put(w http.ResponseWriter, r *http.Request) error {
-	ifNoneMatch := ConditionalMatch(r.Header.Get("If-None-Match"))
+	fi, created, err := b.FileSystem.Create(r.Context(), r.URL.Path, r.Body)
-	ifMatch := ConditionalMatch(r.Header.Get("If-Match"))
-
-	opts := CreateOptions{
-		IfNoneMatch: ifNoneMatch,
-		IfMatch:     ifMatch,
-	}
-	fi, created, err := b.FileSystem.Create(r.Context(), r.URL.Path, r.Body, &opts)
 	if err != nil {
 		return err
 	}

webdav.go

@@ -19,11 +19,6 @@ type FileInfo struct {
 	ETag     string
 }
 
-type CreateOptions struct {
-	IfMatch     ConditionalMatch
-	IfNoneMatch ConditionalMatch
-}
-
 type CopyOptions struct {
 	NoRecursive bool
 	NoOverwrite bool
@@ -53,11 +48,3 @@ func (val ConditionalMatch) ETag() (string, error) {
 	}
 	return string(e), nil
 }
-
-func (val ConditionalMatch) MatchETag(etag string) (bool, error) {
-	if val.IsWildcard() {
-		return true, nil
-	}
-	t, err := val.ETag()
-	return t == etag, err
-}