Skip to content

Commit 7bb77df

Browse files
committed
Include node details when the node is new
In cases where we only receive node details for the first time after the last-seen timestamp, we were mistakenly not including the node info at all in the snapshot. This was the most egregious for initial syncs, where we'd include no node info at all. Luckily the fix is simple, just include a full node info sync in such cases.
1 parent b414c4d commit 7bb77df

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/serialization.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,10 @@ pub(super) fn serialize_delta_set(channel_delta_set: DeltaSet, node_delta_set: N
240240
}
241241
Some((id, delta))
242242
} else {
243-
None
243+
// If the node details didn't exist before the last-update timestamp, send a full
244+
// update.
245+
delta.strategy = Some(NodeSerializationStrategy::Full)
246+
Some((id, delta))
244247
}
245248
}).collect();
246249

0 commit comments

Comments
 (0)