Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: add debug logs to OverlayService::find_enr #1600

Merged
merged 1 commit into from
Dec 7, 2024

Conversation

morph-dev
Copy link
Collaborator

What was wrong?

It seems that OverlayService::find_enr sometimes finds wrong Enr.

See #1596 (comment) for more details

How was it fixed?

This isn't a fix. I'm just adding extra logging to confirm my suspicious and try to find out the root cause of the problem.

To-Do

@morph-dev morph-dev added bug Something isn't working investigation-diary A tale of a journy to vanquish a bug labels Dec 7, 2024
@morph-dev morph-dev self-assigned this Dec 7, 2024
Copy link
Member

@KolbyML KolbyML left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@morph-dev morph-dev merged commit c1c6a72 into ethereum:master Dec 7, 2024
9 checks passed
@morph-dev morph-dev deleted the find_enr_logs branch December 7, 2024 20:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working investigation-diary A tale of a journy to vanquish a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants