trufnetwork / trufnetwork/node
Problem: Anyone can sync with TSN nodes
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 7
- Forks
- 3
- Avg merge
- 3h 2m
- Merged PRs (30d)
- 13
Description
What is happening:
- before node operators, our p2p port was closed.
- with node operators, we need to give nodes p2p access
- currently, anyone with p2p access can say, "I want to be a read-only node of TSN chain" even if they are not validators
- this means, when they connect, their DB is filled with all current data, without needing an approval
Solutions:
- make VPN part of our infra, guide node operators to connect through it
- ask the kwil team to add an option that disallows syncing without being a validator (1 week work, Brennan said). This makes the read-only nodes not possible, correct?
- keep it insecure for now, relying on the obscurity that our
tsn-node-operatorscan remain private for now, and people won't immediately know the servers, genesis file, software, etc(@brennanjl all correct?)
I'd vote for 2, or incrementally 3, then 2. Reasons:
- onboarding won't be blocked
1makes the infra + onboarding significantly more complex3seems ok with Cameron's decision about priority- VPN would also be a temporary solution, requiring more trust in operators, etc
Originally posted by @outerlook in https://github.com/truflation/tsn/issues/457#issuecomment-2288518928
@markholdex
@outerlook let's go with 3 now until 2 is not ready. Isolate no 2 in a separate problem, please.
@brennanjl What are the limitations or negative consequences of option 2? When can you put it on your roadmap and when can we see it ready?
Contributor guide
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
The issue names no files, tests, or entry points. Start by tracing how TSN/Kwil nodes authorize peer access and synchronization, then confirm the chosen VPN or validator-only approach with maintainers. Done means unauthorized peers cannot populate a read-only node with chain data and the operational guidance reflects the decision.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- authorization, distributed-systems, networking
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100