GUI / nginx-upstream-dynamic-servers

An nginx module to resolve domain names inside upstreams and keep them up to date.
MIT License
311 stars 74 forks source link

Nginx 1.10 compatibility #23

Closed milgner closed 7 years ago

milgner commented 7 years ago

Right now this module is unfortunately incomptabible with version 1.10 of Nginx.

../nginx-upstream-dynamic-servers-0.4.0/ngx_http_upstream_dynamic_servers.c: In function ‘ngx_http_upstream_dynamic_server_resolve_handler’:
../nginx-upstream-dynamic-servers-0.4.0/ngx_http_upstream_dynamic_servers.c:421:20: error: assignment from incompatible pointer type [-Werror]

The problem here is that they introduced a separate type ngx_resolver_addr_t that is incompatible to ngx_addr_t. I don't have any experience with the Nginx source so I'm not sure whether there are any quick conversion functions or what needs to be done here...

wandenberg commented 7 years ago

@milgner this was fixed on this commit 29e05c5de Use the master branch, please.

waitspring commented 2 years ago

Did this commit has been published in a new release?