krrishmahar commented on issue #15935: URL: https://github.com/apache/dubbo/issues/15935#issuecomment-3712238950
After reading through `NetUtils` and `NetUtilsTest` in more detail, I noticed that while IPv6 is handled in parts (validation, normalization), the overall behavior—especially around `matchIpExpression` (line 754 in NetUtils.java) —is not explicitly defined or tested for IPv6. I’d like to first focus on adding clear, CI-safe test cases around IPv6 (including edge cases and normalization), and explicitly define the expected behavior for IPv6 in matchIPExpression (even if that behavior is “unsupported” in some system). This should help make the current design more robust and avoid ambiguous behavior when using in dual-stack or IPv6-only environments. I’m a new contributor, so I’d really appreciate any guidance on whether this direction makes sense before I start working on it. -- 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]
