Fix attachment pruning logging; .mp4 extension issue

server/file_cache.go

@@ -71,13 +71,12 @@ func (c *fileCache) Write(id string, in io.Reader, limiters ...*util.Limiter) (i
 }
 
 func (c *fileCache) Remove(ids ...string) error {
-	var firstErr error
 	for _, id := range ids {
-		if err := c.removeFile(id); err != nil {
+		if !fileIDRegex.MatchString(id) {
-			if firstErr == nil {
+			return errInvalidFileID
-				firstErr = err // Continue despite error; we want to delete as many as we can
-			}
 		}
+		file := filepath.Join(c.dir, id)
+		_ = os.Remove(file) // Best effort delete
 	}
 	size, err := dirSize(c.dir)
 	if err != nil {
@@ -86,15 +85,7 @@ func (c *fileCache) Remove(ids ...string) error {
 	c.mu.Lock()
 	c.totalSizeCurrent = size
 	c.mu.Unlock()
-	return firstErr
+	return nil
-}
-
-func (c *fileCache) removeFile(id string) error {
-	if !fileIDRegex.MatchString(id) {
-		return errInvalidFileID
-	}
-	file := filepath.Join(c.dir, id)
-	return os.Remove(file)
 }
 
 func (c *fileCache) Size() int64 {

util/util.go

@@ -173,6 +173,8 @@ func ExtensionByType(contentType string) string {
 	switch contentType {
 	case "image/jpeg":
 		return ".jpg"
+	case "video/mp4":
+		return ".mp4"
 	default:
 		exts, err := mime.ExtensionsByType(contentType)
 		if err == nil && len(exts) > 0 && extRegex.MatchString(exts[0]) {