Warn when non-default TUF mirror URL is configured

Abierto
#1,665 1 comentario 0 reacciones 1 asignado Ver en GitHub

Los mantenedores suelen responder en 2 días

@anithapriyanatarajan ya está trabajando en esto.

Desde el 28/5/2026.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

kind/feature

Summary

When signers.x509.tuf.mirror.url is set to a non-default value, Chains should log a prominent warning that the trust boundary has been expanded. This setting controls where the TUF root metadata (root.json) is fetched from — effectively a trust-root selector.

The setting exists for legitimate air-gapped deployments with local Fulcio + TUF mirrors, but operators should be aware of the security implications.

Proposal

  1. Log a warning at startup when signers.x509.tuf.mirror.url is non-default — e.g.: "Custom TUF mirror URL configured. The TUF root will be fetched from [URL]. This setting is intended for air-gapped environments with a local Fulcio deployment and TUF mirror."
  2. Add URL parsing / sanity checks — reject obviously invalid URLs, log scheme/host for visibility
  3. Document in the security/trust model docs that this setting expands the trust boundary and is meant for air-gapped environments only

Context

Reported via GHSA-jqc2-g622-rrfg. The setting passes the URL to http.Get() without validation (pkg/chains/signing/x509/x509.go:169). While exploitation requires write access to the chains-config ConfigMap (which already grants broad control over Chains behavior), surfacing a warning makes misconfiguration or compromise more visible to operators.

Related: #1661 (trust model documentation)

/kind feature

Lenguaje dominante
Go
Estrellas
279
Forks
171
Merge medio
2 d 4 h
PR fusionados (30 d)
70

Preparar el entorno

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 tektoncd/chains

Todos los issues de tektoncd/chains

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.