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

Aberta Para iniciantes
#20 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
84/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
github-actions, yaml
Domínio
ci-cd, security

Direção de pesquisa

Comece por .github/workflows/deploy.yml e inspecione as permissões do workflow e cada linha uses:. Defina as permissões do workflow como contents: read, conceda permissões de Pages e OIDC ao job de deploy e fixe as Actions referenciadas em SHAs completos de commit, com comentários de versão. Execute npm run typecheck e npm run build; o trabalho estará concluído quando o workflow atender aos critérios indicados de permissões e fixação e um push para main continuar fazendo o deploy.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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

Linguagem predominante
TypeScript
Estrelas
3
Forks
3
Métricas de merge de PRs
Nenhum PR com merge em 30d

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de TMHSDigital/GH-Arcade

Todas as issues de TMHSDigital/GH-Arcade

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.