klesh opened a new pull request, #9043: URL: https://github.com/apache/devlake/pull/9043
## Summary - Renamed `GetURIStringPointer` to `ResolveRequestURI` and added a doc comment explaining its resolution semantics. - Added validation that `relativePath` must be a relative reference: `url.URL.ResolveReference` silently discards the base URL when the reference is absolute (`scheme://...`) or protocol-relative (`//host/...`), which would send the request to the wrong host with no error. Now this returns a `BadInput` error instead. - Updated both call sites (`ApiClient.Do` and the GitHub plugin's `refreshApiClient.do`). ## Test plan - [x] `go build ./helpers/pluginhelper/api/... ./plugins/github/token/...` passes - [x] No other callers or tests reference the old function name (verified via repo-wide search) 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
