feat(kafka-logger): add TLS support for Kafka brokers#13607
Conversation
|
Hi, thank you for your contribution. I looked through the historical APISIX issues related to this topic and agree that this is a reasonable fix. A previous issue, #9353, was closed by the bot as stale. Also, services such as AWS MSK require TLS, which means So I think this is a valid request. |
|
@ecsimsw some CI fails - do you have time to fix it before further reviewing? |
|
@juzhiyuan I've checked the CI failures. The last test environment already has Kafka SSL configured.
Which approach would you prefer? ** |
|
Hi @juzhiyuan, just a friendly follow-up — CI is green now. |
|
Hi @ecsimsw, thanks for your following up! I will request reviews from team next monday. |
Description
Changes
Which issue(s) this PR fixes:
Verification
luacheck: passed locally (0 warnings / 0 errors)check-test-code-style.sh: passed locallylj-releng: requires LuaJIT 2.1 (not available locally), to be verified by CIChecklist