celestiaorg / celestiaorg/go-header

sync | p2p: Collect more information about invalid network head in metric when rejecting it

Open
#141 2 comments 0 reactions 0 assignees View on GitHub
good first issue p2p
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.