trufnetwork / trufnetwork/node

Problem: Anyone can sync with TSN nodes

Open
#470 0 comments 0 reactions 0 assignees View on GitHub

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:
  1. make VPN part of our infra, guide node operators to connect through it
  2. 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?
  3. keep it insecure for now, relying on the obscurity that our tsn-node-operators can 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
  • 1 makes the infra + onboarding significantly more complex
  • 3 seems 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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.