[GitHub] [apisix] tzssangglass edited a comment on issue #6021: bug: in Orchestration, OpenID-Connect Plug-in Properties Timeout does not set, there will be an error

[GitBox <gi...@apache.org>]

tzssangglass edited a comment on issue #6021:
URL: https://github.com/apache/apisix/issues/6021#issuecomment-1005860080


   `conf.timeout` has default value, see: https://github.com/apache/apisix/blob/3f66fab63ecffa7bf4d5bdb02ce8a688a4285709/apisix/plugins/openid-connect.lua#L41-L46
   
   can you add a log like this:
   
   ```diff
   diff --git a/apisix/plugins/openid-connect.lua b/apisix/plugins/openid-connect.lua
   index 1c8d78c1..085581c8 100644
   --- a/apisix/plugins/openid-connect.lua
   +++ b/apisix/plugins/openid-connect.lua
   @@ -240,6 +240,7 @@ function _M.rewrite(plugin_conf, ctx)
    
        -- Previously, we multiply conf.timeout before storing it in etcd.
        -- If the timeout is too large, we should not multiply it again.
   +    ngx.log(ngx.WARN, "conf : ", require("inspect")(conf))
        if not (conf.timeout >= 1000 and conf.timeout % 1000 == 0) then
            conf.timeout = conf.timeout * 1000
        end
   ```
   
   And you need to provide more complete reproduction of the steps


-- 
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