fix: set fileSize to full length instead of unread portion (#18409)

Emyrk · web-flow · commit aee96c9eac51 · 2025-06-17T10:57:34.000-05:00
`content.Len()` would return `0` bytes after a file was fully read.
Since the buffer `Len` function returns the length of the unread
portion.
diff --git a/coderd/files/cache.go b/coderd/files/cache.go
@@ -35,7 +35,7 @@ func NewFromStore(store database.Store, registerer prometheus.Registerer, authz
 		return CacheEntryValue{
 			Object: file.RBACObject(),
 			FS:     archivefs.FromTarReader(content),
-			Size:   int64(content.Len()),
+			Size:   int64(len(file.Data)),
 		}, nil
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ func NewFromStore(store database.Store, registerer prometheus.Registerer, authz`
`35`	`35`	`return CacheEntryValue{`
`36`	`36`	`Object: file.RBACObject(),`
`37`	`37`	`FS: archivefs.FromTarReader(content),`
`38`		`- Size: int64(content.Len()),`
	`38`	`+ Size: int64(len(file.Data)),`
`39`	`39`	`}, nil`
`40`	`40`	`}`
`41`	`41`