Scope Pages permissions to the deploy job and pin actions to commit SHAs

Abierto Apto para principiantes
#20 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
84/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
github-actions, yaml
Área
ci-cd, security

Línea de trabajo

Empieza con .github/workflows/deploy.yml e inspecciona los permisos del workflow y cada línea uses:. Establece los permisos del workflow en contents: read, concede permisos de Pages y OIDC al trabajo de despliegue y fija las Actions referenciadas a SHA de commit completos con comentarios de versión. Ejecuta npm run typecheck y npm run build; el trabajo estará terminado cuando el workflow cumpla los criterios indicados de permisos y fijación, y un push a main siga desplegando.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

good first issue severity: low tech-debt

Summary

Workflow hardening: the build job holds deploy permissions it does not need, and actions are referenced by mutable major tags.

Where

  • .github/workflows/deploy.yml:8-11 (workflow-level pages: write, id-token: write)
  • .github/workflows/deploy.yml:21, :22, :28, :29, :41 (uses: lines)

Suggested fix

  • Set top-level permissions: contents: read and move pages: write and id-token: write under jobs.deploy.permissions.
  • Pin each action to a full commit SHA with the version in a trailing comment.

Acceptance criteria

  • Only the deploy job has pages: write and id-token: write
  • All uses: lines reference a commit SHA
  • A push to main still deploys

Getting started

  • Relevant files: .github/workflows/deploy.yml
  • Run locally: npm install, then npm run dev
  • Checks: npm run typecheck and npm run build (there is no test suite yet)
  • Related docs: README.md

Filed by repo-audit/full-audit at b001f5d. Severity: low. Effort: S. audit-key: security|.github/workflows/deploy.yml|permissions

Lenguaje dominante
TypeScript
Estrellas
3
Forks
3
Métricas de merge de PR
Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de TMHSDigital/GH-Arcade

Todos los issues de TMHSDigital/GH-Arcade

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.