-
Notifications
You must be signed in to change notification settings - Fork 40
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
[TASK] add support for babel networks #71
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bitte entferne das Send
aus ifaceSendUnicast
und ifaceSendMulticast
. Ich denke außerdem, dass wir uns auf insgesamt zwei Einstellungen beschränken können:
interface
für Multicast (und Unicast, wenn nicht gesetzt)interface_unicast
für Unicast-Nachrichten. Wenn nicht gesetzt, wirdinterface
verwendet.
Wie sollen die Antworten auf die Unicast-Pakete ankommen, wenn du dafür keinen Socket öffnest? Ist mir unklar ... noch nicht getestet? |
@@ -74,7 +80,7 @@ func main() { | |||
time.Sleep(delay) | |||
} | |||
|
|||
collector = respond.NewCollector(connections, nodes, config.Respondd.Interface, config.Respondd.Port) | |||
collector = respond.NewCollector(connections, nodes, config.Respondd.InterfaceListen, config.Respondd.InterfaceSendMulticast, config.Respondd.InterfaceSendUnicast, config.Respondd.Port) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wenn das drei unterschiedliche interfaces sind, geht gar nichts mehr? Bitte mal drüber nachdenken.
Ist der PR noch aktuell? Wenn ja, bitte auf den |
Hab gerade versucht zu rebasen - keine Lust mehr ... |
#70 should fix