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

Build with Nginx 1.12.0 and Openssl1.1.0f Failed. #1087

Closed
FqqCS opened this issue Jun 7, 2017 · 5 comments

Comments

@FqqCS
Copy link

commented Jun 7, 2017

Compile error:
/home/xxx/nginx/modules/lua-nginx-module-0.10.9rc5/src/ngx_http_lua_module.c: In function ‘ngx_http_lua_merge_srv_conf’:
/home/xxx/nginx/modules/lua-nginx-module-0.10.9rc5/src/ngx_http_lua_module.c:1022:37: error: passing argument 2 of ‘SSL_CTX_sess_set_get_cb’ from incompatible pointer type [-Werror]
ngx_http_lua_ssl_sess_fetch_handler);
^
In file included from src/event/ngx_event_openssl.h:15:0,
from src/core/ngx_core.h:82,
from /home/xxx/nginx/modules/lua-nginx-module-0.10.9rc5/src/ddebug.h:13,
from /home/xxx/nginx/modules/lua-nginx-module-0.10.9rc5/src/ngx_http_lua_module.c:11:
/openssl-OpenSSL_1_1_0f/.openssl/include/openssl/ssl.h:637:6: note: expected ‘struct SSL_SESSION * (*)(struct ssl_st *, const unsigned char *, int, int )’ but argument is of type ‘struct SSL_SESSION * ()(struct SSL *, u_char *, int, int *)’
void SSL_CTX_sess_set_get_cb(SSL_CTX *ctx,
^
cc1: all warnings being treated as errors

Nginx:1.12.0
lua-nginx-module:lua-nginx-module-0.10.9rc5
openssl:openssl-OpenSSL_1_1_0f

@FqqCS

This comment has been minimized.

Copy link
Author

commented Jun 7, 2017

@agentzh
Obviously, you guys are in compatible with openssl 1.1.x, may openssl 1.1.1 is going to be released, may I ask you a scheduled plan of compatible with openssl 1.1.x? will you guys just skip compatible with openssl 1.1.0 and choose compatible with openssl 1.1.1 instead ?

@FqqCS

This comment has been minimized.

Copy link
Author

commented Jun 7, 2017

@agentzh

This comment has been minimized.

Copy link
Member

commented Jun 9, 2017

@FqqCS See the discussions in #922 and #761. It still needs work. OpenSSL's older series are still updating, like the 1.0.2 series. So there's no rush here at all.

@agentzh

This comment has been minimized.

Copy link
Member

commented Jun 9, 2017

@FqqCS OpenSSL makes many incompatible API changes. That's why it needs a lot of work on our side.

@agentzh

This comment has been minimized.

Copy link
Member

commented Jun 9, 2017

This issue is duplicate with existing issues and pull requests.

@agentzh agentzh closed this Jun 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.