crazyMonkey1995 commented on code in PR #6686: URL: https://github.com/apache/apisix/pull/6686#discussion_r868715860
########## apisix/plugins/redirect.lua: ########## @@ -147,6 +148,7 @@ function _M.rewrite(conf, ctx) core.log.info("plugin rewrite phase, conf: ", core.json.delay_encode(conf)) local ret_code = conf.ret_code + local ret_port = tonumber(ctx.var["var_x_forwarded_port"]) Review Comment: In my opinion, it is definitely impossible to know the listening port of https for this domain name if it is not configured from http to https, so it can be considered that the port can be configured, but the default is 443? -- 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-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org