crashappsec / chalk

Chalk allows you to follow code from development, through builds and into production.
https://crashoverride.com/
GNU General Public License v3.0
322 stars 11 forks source link

fix(git): handle branch names with / #340

Closed miki725 closed 3 weeks ago

miki725 commented 3 weeks ago

Issue

chalk does not report COMMIT_ID for branches with / in them

Description

If the branch is "foo/bar", chalk incorrectly split the branch name and therefore was only attempting to find commit id for branch "foo" which is incorrect.

Testing

➜ make tests args="test_git.py::test_repo --logs -x"