fixup downloader

ethanndickson · ethanndickson · commit e6208e8acac0 · 2025-01-08T15:24:11.000+11:00
diff --git a/Coder Desktop/VPNLib/Downloader.swift b/Coder Desktop/VPNLib/Downloader.swift
@@ -117,10 +117,7 @@ public actor Downloader {
         var req = URLRequest(url: src)
         if FileManager.default.fileExists(atPath: dest.path) {
             if let existingFileData = try? Data(contentsOf: dest) {
-                let sha1Hash = Insecure.SHA1.hash(data: existingFileData)
-                let etag = sha1Hash.map { String(format: "%02x", $0) }.joined()
-                // ETag header needs to be quoted
-                req.setValue("\"\(etag)\"", forHTTPHeaderField: "If-None-Match")
+                req.setValue(etag(data: existingFileData), forHTTPHeaderField: "If-None-Match")
             }
         }
         // TODO: Add Content-Length headers to coderd, add download progress delegate

Original file line number	Diff line number	Diff line change
`@@ -117,10 +117,7 @@ public actor Downloader {`
`117`	`117`	`var req = URLRequest(url: src)`
`118`	`118`	`if FileManager.default.fileExists(atPath: dest.path) {`
`119`	`119`	`if let existingFileData = try? Data(contentsOf: dest) {`
`120`		`- let sha1Hash = Insecure.SHA1.hash(data: existingFileData)`
`121`		`- let etag = sha1Hash.map { String(format: "%02x", $0) }.joined()`
`122`		`- // ETag header needs to be quoted`
`123`		`- req.setValue("\"\(etag)\"", forHTTPHeaderField: "If-None-Match")`
	`120`	`+ req.setValue(etag(data: existingFileData), forHTTPHeaderField: "If-None-Match")`
`124`	`121`	`}`
`125`	`122`	`}`
`126`	`123`	`// TODO: Add Content-Length headers to coderd, add download progress delegate`