[REQ][swift6] Don't pretty-print JSON request bodies
Los mantenedores suelen responder en 1 día
Ya se ha fusionado un pull request relacionado.
- #25193 de @spigo — fusionado
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
Línea de trabajo
El JSONEncoder predeterminado en el CodableHelper de swift6 establece outputFormatting = .prettyPrinted; busca esa línea en las plantillas de swift6 y elimínala. Los DateFormatTests de los ejemplos de swift6 ya quitan .prettyPrinted antes de comparar la salida, así que comprueba si todavía necesitan ese workaround después del cambio. Está terminado cuando un cuerpo de solicitud generado con payload JSON es compacto y las pruebas de los ejemplos de swift6 pasan.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Bug Report Checklist
- Have you provided a full/minimal spec to reproduce the issue?
- Have you validated the input using an OpenAPI validator? (not spec-specific)
- Have you tested with the latest master to confirm the issue still exists?
- Have you searched for related issues/PRs?
- What's the actual output vs expected output?
Description
CodableHelper's default JSONEncoder sets outputFormatting = .prettyPrinted, so every JSON request body is sent with newlines and indentation: extra bytes on every request, no benefit to servers. The samples' own DateFormatTests removes .prettyPrinted before comparing encoded output.
openapi-generator version
master (ba2d6aa3), also 7.26.0. Not a regression.
OpenAPI declaration file content or url
Not spec-specific; reproducible with the petstore spec used by the swift6 samples (modules/openapi-generator/src/test/resources/2_0/swift/petstore-with-fake-endpoints-models-for-testing.yaml).
Generation Details
Any swift6 config.
Steps to reproduce
Send any request with a JSON body and inspect it. Actual: pretty-printed JSON. Expected: compact JSON.
Related issues/PRs
None found.
Suggest a fix
Drop .prettyPrinted from the default encoder; a custom jsonEncoder is unaffected. PR follows.
- Lenguaje dominante
- Java
- Estrellas
- 26.8k
- Forks
- 7.7k
- Merge medio
- 1 d 15 h
- PR fusionados (30 d)
- 145
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- 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 OpenAPITools/openapi-generator
-
[BUG][RUST] chrono date-time path/query/header params are not serialized as RFC 3339Posiblemente ocupada @teovillanueva la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
OpenAPITools/openapi-generator#25297 ·
Los mantenedores suelen responder en 1 día
-
[BUG][RUST] Reqwest - encoding of spaces in path parametersPosiblemente ocupada @GlenWright la tomó hace 4 días. AbiertoIssue: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
OpenAPITools/openapi-generator#25257 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
OpenAPITools/openapi-generator#25237 ·
Los mantenedores suelen responder en 1 día
-
[BUG] OpenApiEvaluator reports "default value not in enum" for numerically equal values with different scale (e.g. 1.5 and 1.50)Posiblemente ocupada @mohitduhan19 la tomó hace 6 días. AbiertoIssue: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
OpenAPITools/openapi-generator#25233 ·
Los mantenedores suelen responder en 1 día
-
[BUG] OpenApiEvaluator emits "Unused model" warnings even when recommendations are disabledPosiblemente ocupada @mohitduhan19 la tomó hace 6 días. AbiertoIssue: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
OpenAPITools/openapi-generator#25232 ·
Los mantenedores suelen responder en 1 día
Todos los issues de OpenAPITools/openapi-generator
Issues similares
-
[Bug] dubbo.protocol.triple.max-message-size is only read from OS environment variablesPosiblemente ocupada @CCCLL la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
hierynomus/sshj#1070 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
quarkiverse/quarkus-jooq#324 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
apache/rocketmq-dashboard#7039 ·
Los mantenedores suelen responder en 3 días
-
pdfium-drift
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
Stirling-Tools/JPDFium#176 ·
Los mantenedores suelen responder en 1 día