Johannes Sixt wrote:
Andreas Ericsson schrieb:+/* + * strip username information from the url + * This will allocate a new string, or return its argument + * if no stripping is necessary. + * + * The url's we want to catch are the following: + * ssh://[user@]host.xz[:port]/path/to/repo.git/ + * [user@]host.xz:/path/to/repo.git/ + * http[s]://[user[:password]@]host.xz/path/to/repo.git + * + * Although git doesn't currently support giving the password + * to http url's on the command-line, it's easier to catch + * that case too than it is to cater for it specially. + */ +static char *anonymize_url(const char *url) +{ + char *anon_url; + const char *at_sign = strchr(url, '@'); + size_t prefix_len = 0; + + if (!at_sign)if (!at_sign || has_dos_drive_prefix(url)) or even better move this function to transport.c and use is_local().
Good idea, even though it really shouldn't matter in practice due to the last if() below. Then again, there's no telling what some silly IDE might take into its head to name paths ;-) I also see now I botched the job and sent the un-amended patch. v2 incoming.
+ return strdup(url); + + if (!prefixcmp(url, "ssh://")) + prefix_len = strlen("ssh://"); + else if (!prefixcmp(url, "http://";)) + prefix_len = strlen("http://";); + else if (!prefixcmp(url, "https://";)) + prefix_len = strlen("https://";); + else if (!strchr(at_sign + 1, ':')) + return strdup(url); +
-- Andreas Ericsson andreas.ericsson@xxxxxx OP5 AB www.op5.se Tel: +46 8-230225 Fax: +46 8-230231 Considering the successes of the wars on alcohol, poverty, drugs and terror, I think we should give some serious thought to declaring war on peace. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html