action: handle slashes in ref names (#63)

* action: handle slashes in ref names Fixes #62. Signed-off-by: William Woodruff <william@trailofbits.com> * requirements: bump sigstore Signed-off-by: William Woodruff <william@trailofbits.com> --------- Signed-off-by: William Woodruff <william@trailofbits.com>
sigstore · Apr 24, 2023 · 64c04b5 · 64c04b5
1 parent 9247cf3
commit 64c04b5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/action.py b/action.py
@@ -62,7 +62,12 @@ def _download_ref_asset(ext):
  repo = os.getenv("GITHUB_REPOSITORY")
  ref = os.getenv("GITHUB_REF")
 
- artifact = Path(f"/tmp/{os.getenv('GITHUB_REF_NAME')}.{ext}")
+ # NOTE: Branch names often have `/` in them (e.g. `feat/some-name`),
+ # which would break the artifact path we construct below.
+ # We "fix" these by lossily replacing all `/` with `-`.
+ ref_name_normalized = os.getenv("GITHUB_REF_NAME").replace("/", "-")
+
+ artifact = Path(f"/tmp/{ref_name_normalized}.{ext}")
 
  # GitHub supports /:org/:repo/archive/:ref<.tar.gz|.zip>.
  r = requests.get(f"https://github.com/{repo}/archive/{ref}.{ext}", stream=True)

diff --git a/requirements.txt b/requirements.txt
@@ -1,2 +1,2 @@
-sigstore ~= 1.1
+sigstore ~= 1.1.2
 requests ~= 2.28