feature: enhance ssl.get_shared_ssl_ciphers() for stream #507
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summy
This pull request introduces a new feature to retrieve shared SSL ciphers in the
ngx_stream_lua_module
and includes corresponding tests. The main changes involve adding a new FFI function, updating the Lua API, and implementing a new test case to validate the feature.Related PR
Closed #505
openresty/lua-nginx-module#1962
openresty/lua-nginx-module#2424
openresty/stream-lua-nginx-module#378
New Feature: Shared SSL Ciphers Retrieval
FFI Function Addition:
ngx_stream_lua_ffi_req_shared_ssl_ciphers
inlib/ngx/ssl.lua
to retrieve shared SSL ciphers. This function includes an option to filter GREASE values.Lua API Update:
ngx_lua_ffi_req_shared_ssl_ciphers
in the Lua API for thengx_stream_lua_module
.Testing Enhancements
Test Plan Update:
t/stream/ssl.t
to accommodate the new test case.New Test Case:
TEST 29
) int/stream/ssl.t
to validate the retrieval of shared SSL ciphers. This test ensures the function works correctly by logging the shared cipher IDs and verifying the output in the error log.