timothymiller / cloudflare-ddns

🎉🌩️ Dynamic DNS (DDNS) service based on Cloudflare! Access your home network remotely via a custom domain name without a static IP!
https://timknowsbest.com/free-dynamic-dns
GNU General Public License v3.0
3.09k stars 322 forks source link

Define better network driver than "host" #158

Open obzocker opened 12 months ago

obzocker commented 12 months ago

Is your feature request related to a problem? Please describe. Currently the readme.md defines a example docker-compose with the docker network driver "host". Without looking in the code it seems that this programm does not need the complete access to all ports/network.

It would be much better from a security standpoint to define the exact ports which need to be open to minimize the attack vector

Describe the solution you'd like Update the readme to concretely specify the ports which needs to be open and update the docker network driver.

archef2000 commented 11 months ago

THe programm does not need any ports forwarded but to get the host network is needed for the IPv6 as it is explained right under the docker-compose