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

Add multidomain support #47

Merged
merged 11 commits into from
Sep 24, 2019
Merged

Conversation

TobleMiner
Copy link
Member

This PR adds true multidomain support. This enables support for site-local multicast groups and thus completes support for Gluon 2019. Commandline options are fully backwards compatible with current master.

To use mesh-announce in a multidomain environment with domains 'one', 'two' and 'three' and the batman-interfaces 'bat-one', 'bat-two' and 'bat-three' run mesh-announce with the following parameters:

mesh-announce -d plugins -i bat-one -b bat-one -i bat-two -b bat-two -i bat-three -b bat-three

mesh-announce will automatically detect the origin domain of requests and answer with data for the correct domain.

This commit allows a single instance of mesh-announce to service a multi-domain environment.
This step is necessitated by a change in gluon 2019 requiring binding to a site-local multicast group.
Previous to this commit -i options could be specified only once per system interface.
Terminating `..]` was missing
@Manawyrm
Copy link

Manawyrm commented Sep 7, 2019

✔️ Using this branch with Freifunk Hannover successfully with seperate domains and their respective batman interfaces. Works!

@kpanic23
Copy link

Just tested (and expanded a little bit), seems to work flawlessly! Thanks a lot!
https://github.com/ff3l/mesh-announce/commits/master

@TobleMiner
Copy link
Member Author

@kpanic23 Thanks for testing! Since I have two independent reports of it working I'll merge this now.

@TobleMiner TobleMiner merged commit fc2d8d7 into ffnord:master Sep 24, 2019
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