GoReleaser version floats on "~> v2" in the job that signs releases
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 66/100
Piste de recherche
Commence dans .github/workflows/go.yml : les étapes goreleaser/goreleaser-action aux lignes 249 (snapshot) et 331 (release) utilisent version: "~> v2", et les permissions du job de release se trouvent autour des lignes 286-289. Choisis une version exacte de GoReleaser et vérifie dans le log de release la version actuellement résolue. C'est terminé lorsque les deux étapes nomment cette version, que le dépôt documente comment la mettre à jour, et que le checkout du job de release définit persist-credentials: false. La méthode de mise à jour relève d'un choix des mainteneurs, donc confirme-la avant de modifier quoi que ce soit.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary
Every action and build tool in the workflows is pinned except GoReleaser itself, which resolves to the newest 2.x at run time in both the snapshot and the release job.
Where
.github/workflows/go.yml:331.github/workflows/go.yml:249.github/workflows/go.yml:286
Details
Every action is pinned to a commit SHA, and golangci-lint (v2.15.0, go.yml:117), govulncheck (@v1.9.0, go.yml:123), syft and cosign (fixed by their pinned installer actions; the log shows syft v1.54.0) are fixed versions. GoReleaser itself is not: goreleaser/goreleaser-action@a4f02ab... # v7.3.0 is given version: "~> v2" at go.yml:249 (snapshot) and go.yml:331 (release). The release-snapshot log of run 43648865601 shows it resolved at run time: Downloading https://github.com/goreleaser/goreleaser/releases/download/v2.20.2/goreleaser_Linux_x86_64.tar.gz, checked only against checksums.txt from the same release. The release job holds contents: write, id-token: write and attestations: write (go.yml:286-289).
Impact
Whatever GoReleaser v2.x is newest on the day of a tag builds, signs and uploads the release, with nothing reviewed and no Dependabot PR. A new minor that changes or deprecates a config key fails or alters a release that the snapshot run on the previous push passed, and a compromised upstream release would run with the signing identity. It is the one build tool in the release path that is not reproducible from the repository.
Steps to reproduce
Compare go.yml:331 with the Downloading .../goreleaser/releases/download/v2.20.2/... line in job 132612931653's log.
Suggested fix
Pin an exact version (version: v2.20.2) in both steps, and keep it current by hand or with a Dependabot-visible form (a tools go.mod or a comment-driven updater such as Renovate's regex manager; Dependabot's github-actions ecosystem does not read with: version).
Notes
While here: the release job's checkout can set persist-credentials: false; GoReleaser and gh take the token from the environment.
Acceptance criteria
- Both jobs name an exact GoReleaser version
- There is a documented way to bump it
- The release job's checkout does not persist credentials
Getting started
- Relevant files:
.github/workflows/go.yml - How to run tests:
make test(orgo test ./...). Tests must stay hermetic: use-simulate,-simulate-zoneor the local DNS test server, never a live domain. - Related docs:
docs/DEVELOPER_GUIDE.md,docs/ARCHITECTURE.md
Filed by repo-audit/full-audit at d0c56f0. Severity: low. Effort: S. audit-key: dependencies|.github/workflows/go.yml|goreleaser-action.version
- Langage dominant
- Go
- Étoiles
- 1
- Forks
- 1
- Merge moyen
- 6 h 42 min
- PR mergées (30 j)
- 7
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de TMHSDigital/subenum
-
good first issue severity: low tech-debt
Difficulté 2/5 Une demi-journée Accessibilité débutants 75/100
TMHSDigital/subenum#329 ·
Les mainteneurs répondent en général sous 1 jour
-
area: docs documentation good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
TMHSDigital/subenum#327 ·
Les mainteneurs répondent en général sous 1 jour
-
ci good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
TMHSDigital/subenum#323 ·
Les mainteneurs répondent en général sous 1 jour
-
area: output bug good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
TMHSDigital/subenum#319 ·
Les mainteneurs répondent en général sous 1 jour
-
community documentation good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
TMHSDigital/subenum#135 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de TMHSDigital/subenum
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
GillMacedo/godebugdiff#13 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
GillMacedo/racesift#14 ·
-
IsTextFile reads the whole file into memory (OOM on large files in the file watcher)Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
gtsteffaniak/filebrowser#3395 ·
Les mainteneurs répondent en général sous 1 jour
-
effort:s impact:low raised-by:worker risk:low
Difficulté 1/5 Moins d'une heure Accessibilité débutants 70/100
medici-finance/assay#6496 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
kind/bug require/reproduce require/severity
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour