There's currently no ref name validation which can cause some interesting problems. PR forthcoming.
TL;DR: bug_demo.go
git has a few naming stipulations. I didn't actually realize how detailed they were until I starting messing around with this earlier today.
git-check-ref-formatdocs- The Stack Overflow answer I started with
- A Regex101 sandbox with the regex in Go
- https://github.com/src-d/go-git/blob/master/plumbing/reference.go
- https://github.com/src-d/go-git/blob/master/references.go
- https://github.com/src-d/go-git/blob/master/config/refspec.go
These are the files I copypasta'd together for the bug demo.
- https://github.com/src-d/go-git/blob/master/_examples/branch/main.go
- https://github.com/src-d/go-git/blob/master/_examples/checkout/main.go
- https://github.com/src-d/go-git/blob/master/_examples/commit/main.go
- https://github.com/src-d/go-git/blob/master/_examples/revision/main.go
tbh I think I lucked into one of the more fun edge cases. The setup is fairly straight forward. I'm going to describe what I'm doing in terms of git commands but I'm actually using go-git.
- I create a repo using
git init(or open an existing one but I don't recommend that; you will lose files). - I make a commit on
masterto move theHEAD - I establish that
go-gitisn't restricting ref names by creating this one:new-..bad\.branch//name. - I create a new branch using that name.
- I attempt to make another commit but at this point the whole thing is hosed. The current ref,
new-..bad\.branch//name., is a combination of a bunch of things that messgitup.
There were some very interesting side effects that popped up, including losting files in the active directory. The first version is worth poking around at. Aside from quirks, suppose your automation pipeline injects some garbage to the ref name overnight and completely mucks up git processes?
Because this is a huge project, fixing this has to be done surgically with as few changes to the underlying API as possible. That's rough. The best place to start is with ReferenceName, the type that runs this. AFAIK I can tell, given its widespread use as as a function (e.g. plumbing.ReferenceName("my-ref")), the best way to handle it would be to convert it from a string to a func(string) string.
I spent a lot of time on this trying to track down the problem. I also spent a lot of time attempting to fix it in place, which didn't work out as well as I would have liked. I'll be submitting a PR for this ASAP.
I spent a ton of time playing around with benchmarks. I'd love to about how to improve them! I tried to add a fair amount of randomness to the process.
The build pipeline is the best place to scope those.
