> - * We also leave "master" as a special case, since it does not reveal > - * anything interesting. > + * We also leave the default branch name as a special case, since it > + * does not reveal anything interesting. > */ I feel this is a weird thing to do, since you're trying to anonymize the branch name,and now the default branch is identifiable with your config file. For example, if the default branch contains the name of my project/repo then this sounds like a recipe for accidentally sharing it. I feel a better alternative would be to exclude nothing from the anonymization or the proposed default default branch name > - if (!strcmp(refname, "refs/heads/master")) > + if (!default_branch_name) > + default_branch_name = git_default_branch_name(0); > + > + if (!strcmp(refname, default_branch_name)) > return refname; > > strbuf_reset(&anon); > -- > gitgitgadget > -- Matthew Rogers
