nodece opened a new pull request, #1468: URL: https://github.com/apache/pulsar-client-go/pull/1468
Fixes https://github.com/apache/pulsar-client-go/issues/1466 ### Motivation Go 1.26 introduces stricter URL parsing via `url.Parse`, rejecting malformed URLs with multiple colons in the host, such as `http://::1/` or `http://localhost:80:80/`. Bracketed IPv6 addresses, e.g., `http://[::1]/`, remain valid. The old behavior can be restored with `GODEBUG=urlstrictcolons=0`. This change exposed redundancy and limitations in the Pulsar client's URL handling, especially for multi-host configurations and internal service resolution. ### Modifications The existing `ServiceNameResolver` is now fully leveraged. Previously, both a `url.URL` object and a `ServiceNameResolver` (which already contains the parsed URL and related info) were passed around, causing redundancy. After this refactor, only the `ServiceNameResolver` is passed, which is sufficient for internal processing. Service URL handling across the client and internal services (`newClient`, `NewLookupService`, `NewHTTPLookupService`) is streamlined. Some internal method signatures and interfaces have changed; these updates are confined to the `internal` package and do not affect end users. Go 1.26 is added to the CI matrix, and stricter URL parsing is enforced using `GODEBUG=urlstrictcolons=1`. -- 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]
