Community
Participate
Working Groups
In practice, some connectors have been using IRepositoryElement.getUrl or AbstractRepositoryConnector.getTaskUrl to reference a stable, non-browseable identifier or URI. Unfortunately the framework will try to surface these identifiers in places such as commit message or task details. There is a need for some connectors to be able to give both a stable URI and browseable URL (whether the latter is stable or not). Unfortunately, AbstractRepositoryConnector.getAutheticatedUrl may not suffice for such a use given that it expects to have authentication information we do not want to include as part of a commit message (ie session token). Therefore, in order to maintain compatibility with existing connectors, I propose adding a 3rd method to AbstractRepositoryConnector, getBrowseableUrl() that returns a true, browseable web URL that contains no authentication information.
Proposal: 25698: 433719: Request for AbstractRepositoryConnector.getBrowseableUrl() [I34eeba69] https://git.eclipse.org/r/#/c/25698/ 25699: 433719: Request for AbstractRepositoryConnector.getBrowseableUrl() [I8b19972e] https://git.eclipse.org/r/#/c/25699/
+1. We should consider other names as well: getOpenableUrl, getLocationUrl
We have settled on getBrowserUrl as best conveying the purpose of the method.
The changes have been merged.