yaoweibin / nginx_tcp_proxy_module

add the feature of tcp proxy with nginx, with health check and status monitor
http://yaoweibin.github.com/nginx_tcp_proxy_module
1.64k stars 365 forks source link

Make with nginx 1.11.4 fail #146

Open g0ddest opened 8 years ago

g0ddest commented 8 years ago

Making with lastest version from git in nginx 1.11.4

Patching exceptions

→ patch -p1 < nginx_tcp_proxy_module/tcp.patch                     
patching file src/core/ngx_log.c
Hunk #1 FAILED at 66.
1 out of 1 hunk FAILED -- saving rejects to file src/core/ngx_log.c.rej
patching file src/core/ngx_log.h
Hunk #1 FAILED at 29.
Hunk #2 FAILED at 36.
2 out of 2 hunks FAILED -- saving rejects to file src/core/ngx_log.h.rej
patching file src/event/ngx_event_connect.h
Hunk #1 succeeded at 33 (offset 1 line).
Hunk #2 succeeded at 45 with fuzz 2 (offset 2 lines).

And make exception:

nginx_tcp_proxy_module/ngx_tcp_core_module.c:33:40: error: ‘NGX_CONF_MULTI’ undeclared here (not in a function)
       NGX_TCP_MAIN_CONF|NGX_CONF_BLOCK|NGX_CONF_MULTI|NGX_CONF_NOARGS,

nginx_tcp_proxy_module/ngx_tcp_core_module.c: In function ‘ngx_tcp_core_listen’:
nginx_tcp_proxy_module/ngx_tcp_core_module.c:484:9: error: cannot convert to a pointer type
         if (ngx_memcmp(ls[i].sockaddr + off, u.sockaddr + off, len) != 0) {

In file included from src/core/ngx_core.h:51:0,
                 from nginx_tcp_proxy_module/ngx_tcp_core_module.c:3:
nginx_tcp_proxy_module/ngx_tcp_core_module.c:504:30: error: incompatible type for argument 2 of ‘memcpy’
     ngx_memcpy(ls->sockaddr, u.sockaddr, u.socklen);

image

LeetJoe commented 7 years ago

same problem. Solved? Thanks.

cell13 commented 7 years ago

have a try, use the command patch -p1 < ../nginx_tcp_proxy_module/tcp_1_8.patch but i meet a problem while making nginx. nginx version: nginx-1.11.5 error: src/event/ngx_event_connect.h:51: error: duplicate member ‘check_index’

gongxuewen commented 7 years ago

I also got same issue.

hsoft-yulijia commented 7 years ago

I also got sane issue.on Nginx-1.12.0.

mahuaibo commented 7 years ago

me too.

onizukaspm commented 6 years ago

same issue, does anybody find a solution? please share

mahuaibo commented 6 years ago

This feature is not supported by private networks.

REPTILEHAUS commented 5 years ago

same issue. +1

wwlyzzyorg commented 5 years ago

同样的问题+1

904715872 commented 4 years ago

同样的问题+1

tortoiselala commented 4 years ago

oh, 2020, I get the same issue

hulining commented 2 years ago

oh, 2021, I got the same issue. nginx version: 1.20.2

yaoweibin commented 2 years ago

see the latest master branch, I have updated the tcp.patch for nginx-1.20.2. It should be OK above the version of nginx.1.8.0. Thank you.

zhiheren @.***> 于2021年12月16日周四 14:30写道:

oh, 2021, I got the same issue

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/yaoweibin/nginx_tcp_proxy_module/issues/146#issuecomment-995479526, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABFRGKNIUX4ZEGO5J77BL3URGBQZANCNFSM4CR7MSSQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.