juzhiyuan commented on PR #13607: URL: https://github.com/apache/apisix/pull/13607#issuecomment-4880045721
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, https://github.com/apache/apisix/issues/9353, was closed by the bot as stale. Also, services such as AWS MSK require TLS, which means `kafka-logger` cannot work properly in those environments before this PR is accepted. Also I notice that https://github.com/ecsimsw/apisix/blob/841dbea7a167684e6b9834c33e841ea61693a100/apisix/pubsub/kafka.lua#L83 already supports TLS, while `kafka-logger` does not. So I think this is a valid request. -- 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]
