Closed jmechalas closed 4 years ago
Dear @jmechalas The key "ssl_engine" existed since nginx-0.1.45-RELEASE, here: http://hg.nginx.org/nginx/annotate/b09ee85d0ac8/src/event/ngx_event_openssl.c#l22
Async-nginx need to use this keyword to enable our QAT-Engine, so we remove these codes. You can check the sample configuration file here: https://github.com/intel/asynch_mode_nginx/blob/master/conf/nginx.QAT-sample.conf
Removing the vendor's keyword just because you want to use a keyword with the same name is not a great solution. In fact, I can't think of a worse solution as it breaks the vendor's original functionality.
Why not choose a new/unique keyword?
@jmechalas: In our async Nginx, the original simple ssl_engine directive is deleted and replaced with ssl_engine framework. This change involves the adding of a new core module and does not depend on the loading of qat module. In other words, no matter whether to load the qat module, we can only use the new ssl_engine framework.
Close it since now more update.
nginx 1.16 added a directive with the name "ssl_engine" in the event module to access OpenSSL engines. It has a different syntax than the one in this module, but generally speaking the name conflict is an issue.