certs setup --json should include the DNS records to create, not just the check response
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
Start by reading internal/command/certificates/root.go, especially printDNSOptions, and trace how fly certs setup --json serializes the CheckCertificate response. Follow the setup logic for apex domains, wildcard and custom certificates, and ACME challenges. Done when human output uses the shared typed options and JSON adds dns_options with only the applicable records, while preserving existing response fields.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Follow-up to #4724 / #4775.
fly certs setup --json now prints JSON, but it prints the raw CheckCertificate response, which is the same thing fly certs check --json prints. The setup-specific decisions live only in printDNSOptions (internal/command/certificates/root.go) and never reach the JSON:
- CNAME is hidden for apex domains (a CNAME at the apex is invalid DNS), but
dns_requirements.cnameis still populated in the JSON, so a script can create a broken record. - Whether the ACME challenge CNAME and the ownership TXT record apply (wildcard / custom cert / ACME logic).
So a script still has to reimplement printDNSOptions to know which records to create.
Proposal: compute the options once into a typed struct (records to create, each with type/name/value and why), render the human text from it, and add it to the JSON as a new field (e.g. dns_options) alongside the existing response fields so current consumers don't break.
- Lenguaje dominante
- Go
- Estrellas
- 1.7k
- Forks
- 320
- Merge medio
- 2 d 22 h
- PR fusionados (30 d)
- 61
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de superfly/flyctl
-
feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Loading a YAML app config with a non-string map key panicsPosiblemente ocupada @alexleventer la tomó hace 3 días. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
fly pg create --network follow-ups: launch, fork/restore Flycast IPs, consistent normalizationPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Los mantenedores suelen responder en 1 día
-
fly launch --db mpg fails with "unknown command"Posiblemente ocupada @miguel-fly la tomó hace 3 días. Abiertobug launch ux
superfly/flyctl#5387 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
`tokens create machine-exec`: metadata-only `secrets list` is denied despite the read fallbackAbiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Los mantenedores suelen responder en 1 día
Todos los issues de superfly/flyctl
Issues similares
-
bug raised-by:desk
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
medici-finance/assay#6102 ·
Los mantenedores suelen responder en 1 día
-
hive size:simple
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
akleizor/ccmorph#456 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
radius-project/radius#13744 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
mvanhorn/printing-press-library#2460 ·
Los mantenedores suelen responder en 1 día