collectionFormat 'multi' is not properly handled in generated code
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 62/100
Direzione di ricerca
Inizia in modules/swagger-codegen/src/main/resources/go/client.mustache, nella funzione di gestione dei parametri indicata, quindi confronta la gestione di multi collectionFormat in modules/swagger-codegen/src/main/resources/python/api_client.mustache. Genera un client Go dalla dichiarazione di esempio con swag init e verifica che una query ids con più valori venga inviata con separatori anziché concatenata.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
Query array declared with collectionFormat(multi) is not generated correctly, with ids concatenated without separator.
Swagger-codegen version
main branch
Swagger declaration file content or url
{
"type": "array",
"items": {
"type": "string"
},
"collectionFormat": "multi",
"description": "ids to query for objects",
"name": "ids",
"in": "query"
},
Command line used for generation
swag init
Steps to reproduce
- Make a REST API endpoint which takes a list of ids and declare it with
collectionFormat(multi)
e.g.// @Param ids query []string false "ids to query for objects" collectionFormat(multi) - Generate API client with
swagger-codegenforgolang - Send a request to API with generated API client code and find on server side that ids are concatenated without separators
Related issues/PRs
Not related, but probably in similar area.
https://github.com/swagger-api/swagger-codegen/issues/10963
Suggest a fix/enhancement
Add handling for collectionFormat multi to the following function:
https://github.com/swagger-api/swagger-codegen/blob/master/modules/swagger-codegen/src/main/resources/go/client.mustache#L129
Similar to python
https://github.com/swagger-api/swagger-codegen/blob/master/modules/swagger-codegen/src/main/resources/python/api_client.mustache#L423
- Lingua principale
- Mustache
- Stelle
- 17.8k
- Fork
- 6k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Apri 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 swagger-api/swagger-codegen
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
swagger-api/swagger-codegen#12755 ·
-
security vulnerability
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
swagger-api/swagger-codegen#11942 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
swagger-api/swagger-codegen#11812 · 1 reazione ·
-
security vulnerability
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
swagger-api/swagger-codegen#11594 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
swagger-api/swagger-codegen#10969 ·
Tutte le issue di swagger-api/swagger-codegen
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
external-issue to-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
googleapis/google-cloud-swift#1151 ·
-
SqliteSaver.list(filter=...) silently misses nested metadata values containing non-ASCII text Apertaexternal
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
langchain-ai/langgraph#9074 · 1 commento ·
-
documentation specification
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
openai/openai-openapi#584 ·