Skip to content

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

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

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

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

FqqCS opened this issue Jun 7, 2017 · 5 comments

Comments

@FqqCS
Copy link

FqqCS 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
Copy link
Author

FqqCS 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
Copy link
Author

FqqCS commented Jun 7, 2017

@agentzh
Copy link
Member

agentzh 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
Copy link
Member

agentzh commented Jun 9, 2017

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

@agentzh
Copy link
Member

agentzh commented Jun 9, 2017

This issue is duplicate with existing issues and pull requests.

@agentzh agentzh closed this as completed Jun 9, 2017
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

2 participants