openapi-generators / openapi-generators/openapi-python-client
Minor schema validation error: doesn't complain about invalid default for null and file types
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 2k
- Forks
- 293
- Merge medio
- 34 min
- PR fusionados (30 d)
- 1
Descripción
Describe the bug
NoneProperty.build and FileProperty.build do have logic to detect if a default other than None was provided for a null property, or if any default was provided for a binary string—and we do have unit tests for those. However, in reality NoneProperty.build and FileProperty.build do not actually get called, because property_from_data just calls the initializers directly instead. Therefore if you do something like this, the default is simply ignored.
OpenAPI Spec File
components:
schemas:
MyModel:
properties:
myNullProperty:
type: "null"
default: "definitely not null"
myBinaryString:
type: string
format: binary
default: "???"
Desktop (please complete the following information):
- OS: any
- Python Version: any
- openapi-python-client version: 0.21.6
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.
Línea de trabajo
Empieza rastreando property_from_data y compara sus llamadas directas al inicializador con NoneProperty.build y FileProperty.build, que ya contienen lógica de validación. Ejecuta las pruebas unitarias existentes para esos constructores y añade cobertura que demuestre que los valores predeterminados no válidos de los esquemas OpenAPI proporcionados se rechazan en lugar de ignorarse.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- api, tooling
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 58/100