Proposal: fully switch to signed peer records, deprecate un-signed address announcements
Open
Nobody has claimed this yet.
- Dominant language
- No language data
- Stars
- 1.8k
- Forks
- 320
- Avg merge
- 11d 15h
- Merged PRs (30d)
- 1
Description
We've already supported signed peer records in Identify for the longest time. Signed peer records are superior, because we know they originate from the right peer (which, with AutoNAT v2, should have a pretty good grasp on its addresses).
We should fully switch to using them everywhere. This includes:
- Preferring addresses from signed peer records when dialing new connections, going as far as to ignore all non-signed addresses if we have at least one signed one.
- Using them for peer records in Kademlia.
- Using them in mDNS.
- Finding other places where address records are added to the peerstore and dialed.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing signed peer records in Identify and tracing address use in Kademlia, mDNS, and the peerstore. Done means all relevant address storage and dialing paths use signed records, with non-signed addresses ignored when a signed address is available.
Written by the indexing model from the issue text.
Assessment
- Domain
- distributed-systems, networking
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100