From cb7cea93a2994bf27b1af8bb33eebd307496046a Mon Sep 17 00:00:00 2001 From: Simon Frei Date: Thu, 12 Jun 2025 22:08:21 +0000 Subject: [PATCH] chore(model): remove redundant removal of internal fields in indexsender (#10173) While it doesn't hurt, it's unnecessary since the big protobuf modernisation, that also introduced types separate from the generated ones for internal use. Those fields are already dropped when converting to the wire in protocol. --- lib/model/indexhandler.go | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/lib/model/indexhandler.go b/lib/model/indexhandler.go index 6ba8e34c7..4149721d0 100644 --- a/lib/model/indexhandler.go +++ b/lib/model/indexhandler.go @@ -489,14 +489,8 @@ func prepareFileInfoForIndex(f protocol.FileInfo) protocol.FileInfo { if f.IsReceiveOnlyChanged() { f.Version = protocol.Vector{} } - // The trailer with the encrypted fileinfo is device local, don't send info - // about that to remotes + // The trailer with the encrypted fileinfo is device local, announce the size without it to remotes. f.Size -= int64(f.EncryptionTrailerSize) - f.EncryptionTrailerSize = 0 - // never sent externally - f.LocalFlags = 0 - f.VersionHash = nil - f.InodeChangeNs = 0 return f }