RFR: Accept trailing slash in URIs for git fork.
Erik Helin
ehelin at openjdk.java.net
Tue Apr 14 09:33:31 UTC 2020
On Tue, 14 Apr 2020 09:29:07 GMT, Erik Helin <ehelin at openjdk.org> wrote:
>> Consider a command like this:
>> $ git fork https://github.com/openjdk/skara/
>>
>> This fails with a fairly cryptic error:
>> Apr 09, 2020 11:39:44 AM org.openjdk.skara.network.RestRequest transformBadResponse
>> WARNING: org.openjdk.skara.network.RestRequest$QueryBuilder at 2a17b7b6
>> Apr 09, 2020 11:39:44 AM org.openjdk.skara.network.RestRequest transformBadResponse
>> WARNING: {"message":"Not Found","documentation_url":"https://developer.github.com/v3"}
>> Exception in thread "main" java.lang.RuntimeException: Request returned bad status: 404
>> at org.openjdk.skara.network/org.openjdk.skara.network.RestRequest.transformBadResponse(RestRequest.java:285)
>> at org.openjdk.skara.network/org.openjdk.skara.network.RestRequest.execute(RestRequest.java:325)
>> at org.openjdk.skara.network/org.openjdk.skara.network.RestRequest$QueryBuilder.execute(RestRequest.java:156)
>> at org.openjdk.skara.forge/org.openjdk.skara.forge.github.GitHubRepository.fork(GitHubRepository.java:231)
>> at org.openjdk.skara.cli/org.openjdk.skara.cli.GitFork.main(GitFork.java:278)
>> at org.openjdk.skara.cli/org.openjdk.skara.cli.GitSkara.main(GitSkara.java:186)
>>
>> The cause is the trailing '/', which is normally accepted when accessing the repository through a web browser, or when
>> cloning the repository. This patch changes the git fork to also accept trailing slashes, by removing them before the
>> GitHubRepository repository instance is created. It is not quite clear if this should be solved at some other level,
>> but felt like a better approach to keep the APIs clean (i.e. just plain repository names), and do normalization in the
>> UI tools.
>
> Looks good!
Hi Jan,
thanks for contributing! Just a super minor nit before you integrate this pull request, could you remove the "."
(period) in the title? In general we don't end pull request titles with a period, similar to how we in JBS don't end
bug titles with a period.
Thanks!
Erik
-------------
PR: https://git.openjdk.java.net/skara/pull/575
More information about the skara-dev
mailing list