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.

Sign up for GitHub

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

Jump to bottom

Fix address advertisement bugs #974

Merged
merged 9 commits into from
Jul 7, 2020
Merged

Fix address advertisement bugs #974

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
743d1cf
fix: use all interface addresses when we can't find the default route
Stebalien Jul 2, 2020
a0ce610
fix: don't add advertise unspecified addresses
Stebalien Jul 2, 2020
e03a031
fix: resolve addresses before looking up observed addresses
Stebalien Jul 2, 2020
293929e
fix: only advertise global unicast
Stebalien Jul 2, 2020
2cc1fa6
fix: filter link-local addresses from advertisement
Stebalien Jul 2, 2020
3d57d61
test: fix basic host addr tests
Stebalien Jul 2, 2020
5d7d30d
Fix comment typo
Stebalien Jul 7, 2020
0f8eb7b
address code review and cleanup some edge cases
Stebalien Jul 7, 2020
8cb0350
Fix english.
Stebalien Jul 7, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 8 additions & 3 deletions p2p/host/basic/basic_host.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,12 @@ func (h *BasicHost) updateLocalIpAddr() {
if err != nil {
log.Errorw("failed to resolve local interface addresses", "error", err)
} else {
h.allInterfaceAddrs = ifaceAddrs
for _, addr := range ifaceAddrs {
// Skip link-local addrs, they're mostly useless.
if !manet.IsIP6LinkLocal(addr) {
h.allInterfaceAddrs = append(h.allInterfaceAddrs, addr)
}
}
}

// If we failed to lookup interface addrs.
Expand All @@ -304,11 +309,11 @@ func (h *BasicHost) updateLocalIpAddr() {
// them.
if len(h.filteredInterfaceAddrs) == 0 {
// Add all addresses.
h.filteredInterfaceAddrs = append(h.filteredInterfaceAddrs, ifaceAddrs...)
h.filteredInterfaceAddrs = h.allInterfaceAddrs
Stebalien marked this conversation as resolved.
Show resolved Hide resolved
} else {
// Only add loopback addresses. Filter these because we might
// not _have_ an IPv6 loopback address.
for _, addr := range ifaceAddrs {
for _, addr := range h.allInterfaceAddrs {
if manet.IsIPLoopback(addr) {
h.filteredInterfaceAddrs = append(h.filteredInterfaceAddrs, addr)
}
Expand Down