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

Binding to all IP addresses #944

Closed
martinpeg26 opened this issue Mar 30, 2018 · 9 comments · Fixed by #1085
Closed

Binding to all IP addresses #944

martinpeg26 opened this issue Mar 30, 2018 · 9 comments · Fixed by #1085
Assignees
Milestone

Comments

@martinpeg26
Copy link

Hi, firstly thank you for an amazing bit of software, it's brilliant.
I've been trying to get ha-bridge v5.0.0, 5.1.0 and 5.2.1 to bind to just my Pi3's IP address for the UPnP port 1900 but none of them will, starting them all I can see that 1900 udp is bound to all interfaces (0.0.0.0).

The setting in my config for Use UPNP Address Interface Only is ticked and the UPnP IP address is set to the host's IP.

I also tried to set this by passing in -Dserver.ip on the command line but it was still the same and I see that command line arguments are deprecated anyways.

Any help anyone can give would be very much appreciated.
Kind regards
Martin

@bwssytems
Copy link
Owner

Will take a look

@bwssytems bwssytems self-assigned this Apr 10, 2018
@martinpeg26
Copy link
Author

Hi, did anyone manager to look at this and come up with a solution?
Cheers
Martin

@bwssytems bwssytems added this to the v5.2.2 milestone Aug 7, 2018
@bwssytems
Copy link
Owner

So it eems I need to be more specific with the binding methods for a specific interface....which is found by ip address. Very unintuitive in Java. Will be updated.

@martinpeg26
Copy link
Author

Ok. C is my language of choice and I'd b a bit lost with Java.
Good luck with it but please realise it's not as important as the other tasks you have, it's just that I'd like to have minidlna running on the same RPi 3 as ha-bridge and domoticz.
Thank you again.
Martin

@bwssytems
Copy link
Owner

Already have it worked out.

@bwssytems bwssytems added bug and removed enhancement labels Aug 7, 2018
@martinpeg26
Copy link
Author

That's brilliant, thank you.

@martinpeg26
Copy link
Author

Hi, I'd forgotten about this and just remembered about it now.
Is there any update/fix please?
Thanks
Martin

@bwssytems
Copy link
Owner

Yes, the development branch has the fix, but i am not liking the way I did it. Thinking through a better solution.

@martinpeg26
Copy link
Author

No problem, I'm fine without this at the minute but would be good to get a fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants