cubicdaiya / ngx_dynamic_upstream

Dynamic upstream for nginx
Other
501 stars 108 forks source link

ngx_dynamic_upstream doesn't check if ip address is valid when adding servers #23

Open sunyq opened 7 years ago

sunyq commented 7 years ago

First of all, ngx_dynamic_upstreamis really an awesome module!

During my test with ngx_dynamic_upstreamI just found it doesn't check the validity of an IP address when I add a server for a zone. For example, if I add "10.1.1:8080" to the zone, the nginx process will be stuck, and I can't get response from nginx. I know this is not the correct way to use ngx_dynamic_upstream, but it would be much better to add an "IP address validity" check feature. Thanks!

reiz commented 6 years ago

I don't think that this module has to solve all problems.