fix(filtering): use HEAD request for version storage to match check

1 file · +3 -7

M internal/filtering/updater.go +3 -7


@@ -252,13 +252,9 @@ func (fu *FilterUpdater) downloadUpdate(ctx context.Context, url, lastModified,
252	252		// Calculate content hash
253	253		hash := fmt.Sprintf("%x", sha256.Sum256(currentContent))
254	254
255		-	// Use HTTP headers as version identifier (same as fetchRemoteVersion)
256		-	version := ""
257		-	if etag := resp.Header.Get("ETag"); etag != "" {
258		-	version = etag
259		-	} else if lastMod := resp.Header.Get("Last-Modified"); lastMod != "" {
260		-	version = lastMod
261		-	}
	255	+	// Fetch version with HEAD request (same method as fetchRemoteVersion)
	256	+	// This ensures consistency - HEAD may return different ETag than GET
	257	+	version, _ := fu.fetchRemoteVersion(ctx, url)
262	258
263	259		// Create update object
264	260		update := &FilterUpdate{

 	// Calculate content hash
 	hash := fmt.Sprintf("%x", sha256.Sum256(currentContent))
-	// Use HTTP headers as version identifier (same as fetchRemoteVersion)
+	// Fetch version with HEAD request (same method as fetchRemoteVersion)
-	version := ""
+	// This ensures consistency - HEAD may return different ETag than GET
-	if etag := resp.Header.Get("ETag"); etag != "" {
+	version, _ := fu.fetchRemoteVersion(ctx, url)
-		version = etag
-	} else if lastMod := resp.Header.Get("Last-Modified"); lastMod != "" {
-		version = lastMod
-	}
 	// Create update object
 	update := &FilterUpdate{