qmsk / e2

E2 Client, Tally output
Mozilla Public License 2.0
26 stars 7 forks source link

Fix tally discovery handling #26

Closed SpComb closed 6 years ago

SpComb commented 6 years ago

Continues #25

The tally:Tally was not using client:Options.DiscoverFilter, and would try to connect to XMLPort=0... have it fail on the client:Options.DiscoverOptions instead.

This would still need some improvements to extend the tally:newSource logic of not connecting to slave VPs to also not connect to non-VP devices... now the tally will log errors for each discovery packet, instead of listing the device as a non-connected source. That is going to be too awkward to test without actual devices, though.