Roman Arutyunyan:
On Tue, Feb 09, 2016 at 08:59:42PM +0100, A. Schulze wrote:
Maxim Dounin:
>Changes with nginx 1.9.11
09 Feb 2016
>
> *) Feature: TCP support in resolver.
the rDNS module (https://www.nginx.com/resources/wiki/modules/rdns/) don't
compile anymore
cc -c -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -Wall
-I src/core -I src/event -I src/event/modules -I src/os/unix -I objs -I
src/http -I src/http/modules -I src/http/v2 -I src/mail \
-o objs/addon/nginx-http-rdns-20140411/ngx_http_rdns_module.o \
./nginx-http-rdns-20140411//ngx_http_rdns_module.c
./nginx-http-rdns-20140411//ngx_http_rdns_module.c: In function
'merge_loc_conf':
./nginx-http-rdns-20140411//ngx_http_rdns_module.c:217:89: error:
'ngx_resolver_t' has no member named 'udp_connections'
if (conf->conf.enabled && ((core_loc_cf->resolver == NULL) ||
(core_loc_cf->resolver->udp_connections.nelts == 0))) {
^
make[2]: *** [objs/addon/nginx-http-rdns-20140411/ngx_http_rdns_module.o]
Error 1
"connections" should be used instead of "udp_connectionss"
Roman,
thanks for the hint. The attached patch solve at least the compile error.
(still untested if the module still work)
Andreas
Index: nginx-1.9.11/nginx-http-rdns-20140411/ngx_http_rdns_module.c
===================================================================
--- nginx-1.9.11.orig/nginx-http-rdns-20140411/ngx_http_rdns_module.c
+++ nginx-1.9.11/nginx-http-rdns-20140411/ngx_http_rdns_module.c
@@ -214,7 +214,7 @@ static char * merge_loc_conf(ngx_conf_t
}
#endif
- if (conf->conf.enabled && ((core_loc_cf->resolver == NULL) || (core_loc_cf->resolver->udp_connections.nelts == 0))) {
+ if (conf->conf.enabled && ((core_loc_cf->resolver == NULL) || (core_loc_cf->resolver->connections.nelts == 0))) {
ngx_conf_log_error(NGX_LOG_EMERG, cf, 0, "no core resolver defined for rdns");
return NGX_CONF_ERROR;
}
_______________________________________________
nginx mailing list
[email protected]
http://mailman.nginx.org/mailman/listinfo/nginx
