Skip to content

ci: arreglar pipelines Hotfix y Security (bugs de config) - #10

Merged
beyondnetPeru merged 1 commit into
mainfrom
ci/fix-hotfix-and-security
Jul 26, 2026
Merged

ci: arreglar pipelines Hotfix y Security (bugs de config)#10
beyondnetPeru merged 1 commit into
mainfrom
ci/fix-hotfix-and-security

Conversation

@beyondnetPeru

Copy link
Copy Markdown
Contributor

Cierra los checks rojos restantes de main, todos bugs de configuración de workflow, no hallazgos de seguridad reales.

Hotfix (hotfix.yml)

Se disparaba en cada push a main y "Validate hotfix branch name" hacía exit 1 (main no es hotfix/**) → toda la pipeline en rojo. Fix: quitar main del trigger (solo hotfix/** + dispatch). El build/test/security de main ya lo cubren ci.yml/build.yml/security.yml.

Security (security.yml)

  • Hardcoded Secrets: goreleaser/gitleaks-action ya no existe (fallo de resolución que tumbaba el job) → gitleaks/gitleaks-action@v2 (best-effort con continue-on-error; el gate real es el grep custom, que no matchea nada en src/).
  • Dependency Review: dependency-review-action requiere base/head ref → solo PRs. if: github.event_name == 'pull_request' (en push a main queda skipped, no failure).
  • CodeQL (.NET Backend): el autobuild de C# fallaba (requiere SDK .NET 10 + restore de shells) → build-mode: none (extracción buildless).

El Security Gate solo bloquea con result == failure (no con skipped), así que con estos fixes pasa.

🤖 Generated with Claude Code

Hotfix (hotfix.yml):
- Se disparaba en cada push a `main` y el job "Validate hotfix branch name" hacía
  exit 1 (main no es `hotfix/**`), tumbando toda la pipeline. Se quita `main` del
  trigger: la pipeline Hotfix solo corre en ramas `hotfix/**` (+ dispatch). El
  build/test/security de main ya lo cubren ci.yml/build.yml/security.yml.

Security (security.yml):
- Hardcoded Secrets: `goreleaser/gitleaks-action` ya no existe (fallo de resolución
  → job caído). Se apunta a `gitleaks/gitleaks-action@v2` (best-effort, con
  continue-on-error); el gate real es el grep custom, que no matchea nada.
- Dependency Review: `dependency-review-action` requiere base/head ref → solo
  funciona en pull_request. Se limita a `if: github.event_name == 'pull_request'`
  (en push a main queda skipped, no failure).
- CodeQL (.NET Backend): el autobuild de C# fallaba (requiere SDK .NET 10 + restore
  de shells NuGet). Se usa `build-mode: none` (extracción buildless), suficiente
  para el gate. El Security Gate solo bloquea con result==failure, no con skipped.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@beyondnetPeru
beyondnetPeru merged commit 684f4ab into main Jul 26, 2026
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant