Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

p2p/discover: improved node revalidation #687

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sonhv0212
Copy link
Contributor

@sonhv0212 sonhv0212 force-pushed the discovery-new-reval branch from 30f4351 to b31535a Compare February 6, 2025 09:51
@sonhv0212 sonhv0212 force-pushed the discovery-new-reval branch from b31535a to 856723c Compare February 7, 2025 03:57
@minh-bq minh-bq added this to the v2.9.1 milestone Feb 10, 2025
@minh-bq
Copy link
Collaborator

minh-bq commented Feb 10, 2025

We should copy description from the above commit to this PR.

This PR will revert the effect of this commit: f0935e6 ("p2p/discover: apply enr filtering in discovery phase, fix minor logic in doRevalidate"). So maybe @huyngopt1994 wants to take a look.

@minh-bq minh-bq requested a review from huyngopt1994 February 10, 2025 06:47
@huyngopt1994
Copy link
Collaborator

@minh-bq Yes, @sonhv0212 need to rework this PR again for adapting with the implementing enr filtering .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants