mab122
/
radicle-heartwood-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

node: Fix routing table update on refs 

The announcer is always the one who must have the refs, not the relayer.
This was changed in a recent patch.
This commit is contained in:
Alexis Sellier 2023-03-26 21:19:24 +02:00 committed by Alexis Sellier
parent 7c41c5edff
commit 973f618cd3
No known key found for this signature in database
2 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
radicle-cli/tests/commands.rs
View File

@ -594,8 +594,6 @@ fn rad_workflow() {
)
.unwrap();
bob.connect(&alice).converge([&alice]);
test(
"examples/workflow/3-issues.md",
&working.join("bob").join("heartwood"),
@ -612,8 +610,6 @@ fn rad_workflow() {
)
.unwrap();
bob.converge([&alice]);
test(
"examples/workflow/5-patching-maintainer.md",
&working.join("alice"),

4
radicle-node/src/service.rs
View File

@ -873,14 +873,14 @@ where
// from a new repository being initialized.
if let Ok(updated) = self
.routing
.insert(message.rid, *relayer, message.timestamp)
.insert(message.rid, *announcer, message.timestamp)
{
if updated {
self.emitter.emit(Event::SeedDiscovered {
rid: message.rid,
nid: *relayer,
});
info!(target: "service", "Routing table updated for {} with seed {relayer}", message.rid);
info!(target: "service", "Routing table updated for {} with seed {announcer}", message.rid);
}
}
// Discard announcement messages we've already seen, otherwise update