ecsimsw opened a new pull request, #13607:
URL: https://github.com/apache/apisix/pull/13607

   ### Description
   
   Add `tls` option to `kafka-logger` and `error-log-logger` plugins for 
connecting to TLS-enabled Kafka brokers (e.g. AWS MSK). APISIX uses 
lua-resty-kafka for Kafka produce, and lua-resty-kafka already supports 
`ssl`/`ssl_verify` in broker_config. However, APISIX plugins were not exposing 
these options to users. This PR adds a `tls` object schema so users can enable 
TLS connections. The `tls` object design allows future extension for mTLS 
support.  
   
   ### Changes
   - kafka-logger.lua : add tls schema + pass ssl/ssl_verify to broker_config
   - error-log-logger.lua : same change
   - kafka-logger-tls.t : schema validation + integration tests via SSL port 
(9093)
   - error-log-logger-kafka.t : schema validation + integration tests for TLS
   - kafka-logger.md, error-log-logger.md : add tls attributes and usage 
examples
   
   #### Which issue(s) this PR fixes:
   - Fixes #9353
   
   ### Checklist
   - [x ] I have explained the need for this PR and the problem it solves
   - [x] I have explained the changes or the new features added to this PR
   - [x] I have added tests corresponding to this change
   - [x] I have updated the documentation to reflect this change
   - [x] I have verified that this change is backward compatible (If not, 
please discuss on the [APISIX mailing 
list](https://github.com/apache/apisix/tree/master#community) first)
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to