zaunist commented on a change in pull request #6092:
URL: https://github.com/apache/apisix/pull/6092#discussion_r785583833
##########
File path: apisix/admin/plugin_config.lua
##########
@@ -100,13 +102,33 @@ end
function _M.delete(id)
+ if not id then
+ return 400, {error_msg = "missing plugin config id"}
+ end
+
+ local routes, routes_ver = get_routes()
+ core.log.info("routes: ", core.json.delay_encode(routes, true))
Review comment:
I did it with reference to the processing of upstream. At the same time,
I also considered whether to logging all route information. If it is not log
here, does it also need to be modified in upstream?
https://github.com/apache/apisix/blob/c38e94f20144b0ba01018970fa954c935a325ac1/apisix/admin/upstreams.lua#L130
--
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]
