Skip to content

honor remote_allow_list in hole punch response - #1186

Merged
wadey merged 8 commits into
masterfrom
holepunch-remote-allow-list
Nov 10, 2025
Merged

honor remote_allow_list in hole punch response#1186
wadey merged 8 commits into
masterfrom
holepunch-remote-allow-list

Conversation

@wadey

@wadey wadey commented Jul 23, 2024

Copy link
Copy Markdown
Member

When we receive a "hole punch notification" from a Lighthouse, we send a hole punch packet to every remote of that host, even if we don't include those remotes in our "remote_allow_list". Change the logic here to check if the remote IP is in our allow list before sending the hole punch packet.

When we receive a "hole punch notification" from a Lighthouse, we send
a hole punch packet to every remote of that host, even if we don't
include those remotes in our "remote_allow_list". Change the logic here
to check if the remote IP is in our allow list before sending the hole
punch packet.
nbrownus
nbrownus previously approved these changes Jul 23, 2024
@wadey wadey added this to the v1.10.0 milestone Jul 24, 2024
@wadey wadey changed the title honor remote_allow_ilst in hole punch response honor remote_allow_list in hole punch response Jul 24, 2024
@johnmaguire

Copy link
Copy Markdown
Collaborator

@JackDoan / @nbrownus Do you know if the cert v2 work has made this PR unnecessary?

@wadey Can you fix the merge conflict if not? Thanks!

@wadey

wadey commented Sep 16, 2025

Copy link
Copy Markdown
Member Author

fixed merge conflicts 👍

@wadey
wadey merged commit 97b3972 into master Nov 10, 2025
9 checks passed
@wadey
wadey deleted the holepunch-remote-allow-list branch November 10, 2025 18:52
@nbrownus nbrownus mentioned this pull request Nov 19, 2025
63 tasks
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