lihanghang404 opened a new issue, #8417: URL: https://github.com/apache/apisix/issues/8417
### Current Behavior kafka-logger插件前端校验必须输入broker_list属性,  后端lua脚本校验brokers与broker_list只能同时存在一个 oneOf = { { required = {"broker_list", "kafka_topic"},}, { required = {"brokers", "kafka_topic"},}, } ### Expected Behavior 去掉前端broker_list的正则校验。因为仅brokers可以输入sasl_config安全验证相关配置。应该尽快弃用broker_list属性。 ### Error Logs failed to process entries: failed to send data to Kafka topic: not found topic, brokers: {"10.206.67.198":9092}, context: ngx.timer, client: 10.206.65.132, server: 0.0.0.0:9080 ### Steps to Reproduce 1. run apisix via the Docker images 2. create a router in dashboard 3. configuring the kafka-logger plugin in dashboard 4. configuring both of 'broker_list' and 'brokers' 5. logger error commit 'should only match 1 scheme' 6. configuring only the 'brokers' 7. commit 'should have required property 'broker_list'' by chrome 8. configuring 'broker_list' only 9. it doesn't work because i can not input 'sals_conf' in this properties 'broker_list' ### Environment - 3.0.0 - 3.10.0-1160.el7.x86_64 #1 SMP Mon Oct 19 16:18:59 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux - OpenResty / Nginx version (run `openresty -V` or `nginx -V`): - 3.4.5 - 2.13.0 - default - default -- 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]
