QuantumEntangledAndy / neolink

An RTSP bridge to Reolink IP cameras
GNU Affero General Public License v3.0
297 stars 44 forks source link

Add docker run instructions to README #83

Closed PrplHaz4 closed 1 year ago

PrplHaz4 commented 1 year ago

Add docker run instructions to README, use host networking to avoid explicitly mapping a bunch of ports, and trace logging to document for any users that need to troubleshoot.

QuantumEntangledAndy commented 1 year ago

What was your motivation behind this? I'm not sure about the host mode by default. I don't run it like that myself, instead I supply full address information in my config so it can connect without the broadcasts.

PrplHaz4 commented 1 year ago

What was your motivation behind this? I'm not sure about the host mode by default. I don't run it like that myself, instead I supply full address information in my config so it can connect without the broadcasts.

This note was on the original repo. I started using host networking mainly to rule that out as a source of problems when testing a battery cam (Reolink Go PT).

If you use a battery-powered camera (or other UDP-only camera) you will need to either use --net=host or setup a macvlan for the docker image that supports UDP broadcast. This is because UDP requires that udp broadcast messages are transmitted across the docker network interface, however this is https://github.com/docker/for-linux/issues/637

QuantumEntangledAndy commented 1 year ago

I see. Well I might want to add a note about how to connect. Host mode should be mostly optional now since we can connect to udp without using broadcasts now.

But I'll merge this for now and add it to my todo list :)

QuantumEntangledAndy commented 1 year ago

Thanks for the contribution