You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2021/09/13 17:11:21 UTC

[GitHub] [apisix] tzssangglass commented on a change in pull request #5012: feat(debug-mode): add dynamic debug mode

tzssangglass commented on a change in pull request #5012:
URL: https://github.com/apache/apisix/pull/5012#discussion_r707528522



##########
File path: apisix/debug.lua
##########
@@ -30,6 +29,7 @@ local setmetatable = setmetatable
 local pcall        = pcall
 local ipairs       = ipairs
 local unpack       = unpack
+local inspect      = require "inspect"

Review comment:
       `require("apisix.debug")` in `init.lua` --> `require("ngx.process")` in `debug.lua`, will result in error as `process.lua:5: unsupported subsystem: stream`, see: https://github.com/apache/apisix/runs/3578221555
   
   ngx.process API for stream subsystem is not implemented on openresty 1.19.3.2(but is implemented on 1.19.9.1), so I move `require("ngx.process")` to function.




-- 
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: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org