celestiaorg / celestiaorg/go-header
p2p/exchange: `validateChainID` for every exchange endpoint
Open
good first issue
- Dominant language
- Go
- Stars
- 21
- Forks
- 33
- Avg merge
- 5d 20h
- Merged PRs (30d)
- 7
Description
Right now, we only validateChainID for `GetVerifiedRange` -- we should extend this to every endpoint.
Contributor guide
Research direction
Start in the p2p/exchange endpoint implementation, using GetVerifiedRange as the existing validation example, and identify the other exchange endpoints. Add coverage showing that each endpoint validates the chain ID, then run the relevant exchange tests to confirm all endpoints reject an invalid chain ID.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- blockchain, networking
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100