search

tari-project / tari

The Tari protocol
https://tari.com
BSD 3-Clause "New" or "Revised" License
354 stars 219 forks source link

feat(peersync)!: push local peer record to connected peer on change #6687

Closed sdbondi closed 5 days ago

sdbondi commented 1 week ago

Description

feat(peersync)!: push local peer record to connected peer on change

Motivation and Context

Connected peers should have an up-to-date and shareable address record for peers they are connected to. This PR adds this to the peer sync protocol.

How Has This Been Tested?

Manually - looked at logs to observe it working

What process can a PR reviewer use to test or verify this change?

Given node A connected to node B prior, a node C can request node B from Node A and always get a response.

Breaking Changes

BREAKING CHANGE: peer sync protocol is not compatible with previous versions

github-actions[bot] commented 1 week ago

Test Results (CI)

    3 files    126 suites   9m 39s ⏱️ 1 161 tests 1 161 ✅ 0 💤 0 ❌ 3 483 runs  3 483 ✅ 0 💤 0 ❌

Results for commit d8cc83b4.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 week ago

Test Results (Integration tests)

 2 files  10 suites   13m 55s ⏱️ 23 tests 22 ✅ 0 💤 1 ❌ 24 runs  23 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit d8cc83b4.

:recycle: This comment has been updated with latest results.