membphis commented on PR #13666: URL: https://github.com/apache/apisix/pull/13666#issuecomment-4976510880
[P2] This change regresses single-node `least_conn` upstreams that use an unbracketed IPv6 host. The old single-node fast path used the structured `host` and `port` fields directly, but this PR now sends `least_conn` through the picker, where `transform_node()` builds `::1:1980`. `core.utils.parse_addr()` treats that as an IPv6 address without a port, so the request fails. The fact that multi-node upstreams already have the same bug does not make expanding it to the common single-node path backward compatible. Please make the balancer server key IPv6-safe and add single- and multi-node IPv6 regression coverage. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
