[REQ][swift6] Don't pretty-print JSON request bodies
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #25193 di @spigo — integrata
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
Direzione di ricerca
Il JSONEncoder predefinito nel CodableHelper di swift6 imposta outputFormatting = .prettyPrinted; trova quella riga nei template swift6 e rimuovila. I DateFormatTests degli esempi swift6 rimuovono già .prettyPrinted prima di confrontare l'output, quindi verifica se questo workaround serve ancora dopo la modifica. È completato quando un corpo di richiesta generato con un payload JSON è compatto e i test degli esempi swift6 passano.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Java
- Stelle
- 26.8k
- Fork
- 7.7k
- Merge medio
- 1g 14h
- PR unite (30g)
- 148
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di OpenAPITools/openapi-generator
-
[BUG][RUST] Reqwest - encoding of spaces in path parametersForse già presa @GlenWright l’ha presa 1 giorno fa. ApertaIssue: Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
OpenAPITools/openapi-generator#25257 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
[BUG] ExampleGenerator returns the string "{}" (or null) for `type: object` schemas without `properties`Forse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
OpenAPITools/openapi-generator#25237 ·
I maintainer di solito rispondono entro 1 giorno
-
[BUG] OpenApiEvaluator reports "default value not in enum" for numerically equal values with different scale (e.g. 1.5 and 1.50)Forse già presa @mohitduhan19 l’ha presa 3 giorni fa. ApertaIssue: Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
OpenAPITools/openapi-generator#25233 ·
I maintainer di solito rispondono entro 1 giorno
-
[BUG] OpenApiEvaluator emits "Unused model" warnings even when recommendations are disabledForse già presa @mohitduhan19 l’ha presa 3 giorni fa. ApertaIssue: Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
OpenAPITools/openapi-generator#25232 ·
I maintainer di solito rispondono entro 1 giorno
-
[BUG][KOTLIN] multiplatform/jvm-ktor: `DELETE` without `requestBody` sends `"Content-Type: application/json"` with an empty bodyForse già presa @mohitduhan19 l’ha presa 3 giorni fa. ApertaIssue: Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
OpenAPITools/openapi-generator#25223 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di OpenAPITools/openapi-generator
Issue simili
-
jules
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
open-learning-exchange/myplanet#18199 · 1 commento ·
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
Scheme-less stream URLs throw NullPointerException instead of URISyntaxExceptionForse già presa @blihio l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
pedroSG94/RootEncoder#2218 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
apache/rocketmq-dashboard#6872 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
ADORSYS-GIS/token-status-link#168 ·
I maintainer di solito rispondono entro 2 giorni