Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change for function ngx_sock_ntop() in ngx_http_sticky_misc.c to allow for new Nginx 1.5.8+ API use of function: #1

Open
peterbowey opened this issue Mar 4, 2014 · 0 comments

Comments

@peterbowey
Copy link

An API change to ngx_sock_ntop() occurred in Nginx 1.5.8

A patch to ngx_http_sticky_misc.c is required:

There is an extra parameter required [socklen]. for the ngx_sock_ntop()

            if (digest->data == NULL) {
                        return NGX_ERROR;
            }
+#if defined(nginx_version) && nginx_version >= 1005003
+            digest->len = ngx_sock_ntop(in, sizeof(struct sockaddr_in), digest->data, len, 1);
+#else
            digest->len = ngx_sock_ntop(in, digest->data, len, 1);
+#endif
            return NGX_OK;
            return NGX_OK;
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant