fix: PoC fix for #11781, "connect to apisix failed: missing the port number" #11782
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes #11781
In the event that
ngx.var.server_port
is a blank string (e.g. client request arrived via unix socket instead of TCP port),httpc:connect
is given the listening port from the APISIX config, preventing an error.I have tried to cover the possible ways that node_port can be defined in the config.
function tab_is_array(t)
borrowed from:apisix/apisix/cli/file.lua
Line 47 in a0b1a45
Sorry if my Lua code isn't up to your standards, I'm new to Lua and still finding my feet.
Checklist