Bug found related to Boolean fields (with solution)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 42/100
Línea de trabajo
Inspect jsonschema.lua around line 701 and reproduce the failure with the Boolean oneOf schema and default value shown in the issue. Verify that validator generation handles Boolean defaults without the sformat error, and add or run a regression check if the repository provides one.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I found a bug when using boolean fields in my JSON Schema. My partial JSON Schema which I use to detect the error was the following:
...
"bool_param":{
"oneOf":[{
"type":"boolean"
},{
"enum":["external"]
}],
"default":true,
"description":"Description of bool_param. Set as 'external' if the value is not yet known and thus will be set externally at runtime (only once) depending on e.g. the outcome of a previous action or the outcome of another module."
},
...
When setting it to true or false, I got the following error when validating the schema:
lua: /usr/local/share/lua/5.1/jsonschema.lua:701: bad argument #4 to 'sformat' (string expected, got boolean)
stack traceback:
[C]: in function 'sformat'
/usr/local/share/lua/5.1/jsonschema.lua:701: in function 'generate_validator'
/usr/local/share/lua/5.1/jsonschema.lua:173: in function 'validator'
/usr/local/share/lua/5.1/jsonschema.lua:678: in function 'generate_validator'
/usr/local/share/lua/5.1/jsonschema.lua:173: in function 'validator'
/usr/local/share/lua/5.1/jsonschema.lua:1184: in function 'generate_main_validator_ctx'
/usr/local/share/lua/5.1/jsonschema.lua:1201: in function 'generate_validator'
./task_requirements.lua:501: in function 'load_json_table'
example.lua:35: in main chunk
[C]: ?
The bug was (apparently) easily fixed by changing line 701 of the jsonschema.lua from:
ctx:stmt(sformat(' %s[%s] = %s', ctx:param(1), str_filter(prop), default))
to
ctx:stmt(sformat(' %s[%s] = %s', tostring(ctx:param(1)), str_filter(prop), tostring(default)))
I guess this happens because lua does not convert booleans to strings automatically.
- Lenguaje dominante
- Lua
- Estrellas
- 136
- Forks
- 29
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 api7/jsonschema
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
api7/jsonschema#95 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
api7/jsonschema#105 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 20/100
api7/jsonschema#93 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
api7/jsonschema#92 ·
-
Support pcre2Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
api7/jsonschema#89 ·
Todos los issues de api7/jsonschema
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
cataclysmbn/Cataclysm-BN#10516 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
silverbulletmd/silverbullet#2187 ·
Los mantenedores suelen responder en 2 días
-
clangd_extensions.nvim and alabaster.nvim: "p00f" username now belongs to a different account (repo-jacking risk)Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
AstroNvim/astrocommunity#1802 ·
-
Bug era/hc Miscellaneous Task
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