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

Reply via email to