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

Reintroduce Loopback Device Support #31

Merged
merged 1 commit into from
Nov 19, 2022
Merged

Conversation

oxzi
Copy link
Contributor

@oxzi oxzi commented May 21, 2022

The filterInterfaces function reduces the useable network interfaces
for peer discovery. Due to two changes introduced in v1.6.41 and
v1.6.52, a self-discovery over a loopback device was no longer
possible.

For reasons, a device either holds the loopback flag or the multicast
flag, not both together. This commit reflects this.

Furthermore, some refactoring of the filterInterfaces function was
performed to ease its understandability.

Footnotes

  1. 3ea067d

  2. 78d5e82

The `filterInterfaces` function reduces the useable network interfaces
for peer discovery. Due to two changes introduced in v1.6.4[^0] and
v1.6.5[^1], a self-discovery over a loopback device was no longer
possible.

For reasons, a device either holds the loopback flag or the multicast
flag, not both together. This commit reflects this.

Furthermore, some refactoring of the `filterInterfaces` function was
performed to ease its understandability.

[^0]: 3ea067d
[^1]: 78d5e82
oxzi added a commit to dtn7/dtn7-gold that referenced this pull request May 21, 2022
In 9bc28b4, the modules were bumped
but, as it seems, not checked afterwards. This commits replaces the
wonderful schollz/peerdiscovery module with an upcoming pull request,
posted in <schollz/peerdiscovery#31>.
@schollz
Copy link
Owner

schollz commented Nov 19, 2022

Great! Sorry for the late reply

@schollz schollz merged commit 7ebd99d into schollz:master Nov 19, 2022
@oxzi oxzi deleted the loopback-allow branch November 19, 2022 09:36
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.

2 participants