ChainSafe / ChainSafe/lodestar

Improve logging for gossip messages

Open
#6,554 5 comments 0 reactions 0 assignees View on GitHub
help wanted meta-feature-request prio-medium scope-logging
Dominant language
TypeScript
Stars
1.4k
Forks
483
Avg merge
1d 16h
Merged PRs (30d)
150

Description

### Problem description

We recently encountered a scenario where we faced an error from the beacon API endpoint.

```
error: Failed to send out exit message "PublishError.InsufficientPeers"
```

During the debugging, we could not find any such error been logged anywhere. The `validator-ejector` called the beacon API, which internally called the libp2p-gosshipsub, and that error was raised there and then triggered back to API response, and not logged any where in our code.

### Solution description

We should improve the logging for the gossip messages:

1. Log all invocation any gossip message to any topic
2. Log success or error for any publishing message to any topic

### Additional context

Having those logs on the debug level would be sufficient.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.