feature: enhance ssl.get_shared_ssl_ciphers() for stream#507
Closed
chensunny wants to merge 1 commit intoopenresty:masterfrom
Closed
feature: enhance ssl.get_shared_ssl_ciphers() for stream#507chensunny wants to merge 1 commit intoopenresty:masterfrom
chensunny wants to merge 1 commit intoopenresty:masterfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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_moduleand 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_ciphersinlib/ngx/ssl.luato retrieve shared SSL ciphers. This function includes an option to filter GREASE values.Lua API Update:
ngx_lua_ffi_req_shared_ssl_ciphersin the Lua API for thengx_stream_lua_module.Testing Enhancements
Test Plan Update:
t/stream/ssl.tto accommodate the new test case.New Test Case:
TEST 29) int/stream/ssl.tto 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.