ro4i7 commented on code in PR #9116:
URL: https://github.com/apache/apisix/pull/9116#discussion_r1148373644
##########
apisix/stream/router/ip_port.lua:
##########
@@ -78,9 +78,30 @@ do
local route = item.value
if route.protocol and route.protocol.superior_id then
- -- subordinate route won't be matched in the entry
- -- TODO: check the subordinate relationship in the Admin API
- goto CONTINUE
+ local admin_api = require("apisix.admin").check_http_admin()
+ if not admin_api then
+ ngx.log(ngx.ERR, "failed to get http admin node")
Review Comment:
> Can we use "core.log" instead?
@starsz Sorry, I have been out of the station, so replying late. Yeah
definitely, I have updated it according to it, please check.
--
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]