celestiaorg / celestiaorg/go-header
sync | p2p: Collect more information about invalid network head in metric when rejecting it
- Dominant language
- Go
- Stars
- 21
- Forks
- 33
- Avg merge
- 5d 20h
- Merged PRs (30d)
- 7
Description
Collect information about peer ID [here](https://github.com/celestiaorg/go-header/blob/main/p2p/subscriber.go#L127-L129) when recording rejected msg from header-sub (such as the peerID and error message) (and possibly also record the height of the rejected header).
In syncer, record height + reason for rejection [here](https://github.com/celestiaorg/go-header/blob/main/sync/sync_head.go#L179-L185)
Contributor guide
Research direction
Start in p2p/subscriber.go at lines 127-129 and sync/sync_head.go at lines 179-185, tracing how rejected headers are recorded in metrics. Done means the rejection metrics include the requested peer ID and error message, plus the rejected header height where applicable, and syncer metrics include height and rejection reason.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- networking, observability
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100