RFR: Avoid using getUrl when creating folder names

Erik Duveblad via github.com duke at openjdk.java.net
Fri Aug 23 09:37:42 UTC 2019


On Fri, 23 Aug 2019 09:37:40 GMT, Robin Westberg via github.com <duke at openjdk.java.net> wrote:

> Hi all,
> 
> Please review this change that replaces calls to getUrl with getWebUrl when creating names of folders to clone repositories into. The problem is that getUrl can contain credentials, which getWebUrl does not.
> 
> Best regards,
> Robin
> 
> ----------------
> 
> Commits:
>  - 53ede458:	Avoid using getUrl when creating folder names, as it may contain credentials
> 
> Pull request:
> https://git.openjdk.java.net/skara/pull/77
> 
> Webrev:
> https://webrevs.openjdk.java.net/skara/77/webrev.00
> 
> Patch:
> https://git.openjdk.java.net/skara/pull/77.diff
> 
> Fetch command:
> git fetch https://git.openjdk.java.net/skara pull/77/head:pull/77

This PR has been reviewed by Erik Duveblad via github.com - changes are approved. Review comment:

Looks good, great fix!

PR: https://git.openjdk.java.net/skara/pull/77


More information about the skara-dev mailing list