collectionFormat 'multi' is not properly handled in generated code
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 62/100
Línea de trabajo
Comienza en modules/swagger-codegen/src/main/resources/go/client.mustache, en la función referenciada de manejo de parámetros, y compara después el manejo de multi collectionFormat en modules/swagger-codegen/src/main/resources/python/api_client.mustache. Genera un cliente de Go a partir de la declaración de ejemplo con swag init y verifica que una consulta ids con varios valores se envíe con separadores en lugar de concatenada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Mustache
- Estrellas
- 17.8k
- Forks
- 6k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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 swagger-api/swagger-codegen
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
swagger-api/swagger-codegen#12755 ·
-
security vulnerability
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
swagger-api/swagger-codegen#11942 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
swagger-api/swagger-codegen#11812 · 1 reacción ·
-
security vulnerability
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
swagger-api/swagger-codegen#11594 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
swagger-api/swagger-codegen#10969 ·
Todos los issues de swagger-api/swagger-codegen
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
microsoft/magentic-ui#588 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
alexgorbatchev/simple-ptt#3 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
CorrelAid/formtransform#44 ·