re20052 commented on code in PR #64147:
URL: https://github.com/apache/doris/pull/64147#discussion_r3425351675
##########
fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/LoadAction.java:
##########
@@ -504,14 +505,11 @@ private TNetworkAddress
selectEndpointByRedirectPolicy(HttpServletRequest req, B
}
String reqHost = "";
- String[] pair = reqHostStr.split(":");
- if (pair.length == 1) {
- reqHost = pair[0];
- } else if (pair.length == 2) {
- reqHost = pair[0];
- } else {
+ try {
Review Comment:
This change is only intended to correctly split host and port, including
bracketed IPv6 endpoints.
It does not change the downstream public/private classification behavior.
The IPv6 ULA/link-local handling under redirect-policy: public-private is a
separate issue and is outside the scope of this patch.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
