airqo-platform / airqo-platform/AirQo-api

Require CODEOWNERS review and status checks on protected branches

Abierto
#6,961 0 comentarios 0 reacciones 1 asignado Reclamado por @Psalmz777 Ver en GitHub
Lenguaje dominante
JavaScript
Estrellas
26
Forks
24
Merge medio
5 h 36 min
PR fusionados (30 d)
81

Descripción

## Summary
OpenSSF Scorecard flags:
- **Code-Review: 1/10** — only 2 of 12 recently merged changesets had an approved review
- **Branch-Protection: 6/10** — the `staging` branch requires PRs and 4 approvals, but:
- codeowners review is not required
- no required status checks are configured to merge

## Tasks
- [ ] Add a `CODEOWNERS` file mapping key directories/microservices to responsible reviewers
- [ ] Enable "Require review from Code Owners" on branch protection for `staging`/`main`
- [ ] Add required status checks (CI tests, CodeQL) as merge gates on protected branches
- [ ] Investigate why merged changesets are landing without a recorded approval — confirm branch protection is actually enforced org-wide and not bypassable by admins

## Why this matters
Enforced review is one of the highest-leverage Scorecard checks and directly reduces the risk of an unreviewed change introducing a vulnerability or backdoor.

## References
- Scorecard checks: `Code-Review` (score 1), `Branch-Protection` (score 6)

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.