Missing call and argument validation: `{{empty()}}` and `{{ 1|replace }}` render where Jinja2 raises `TypeError`
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 74/100
Línea de trabajo
Start in src/expression_evaluator.cpp, following the evaluator paths that dispatch calls and builtin filters. Reproduce {{empty()}} and {{ 1|replace }} from the issue, then verify that both invalid cases produce render errors rather than empty output, while valid calls and filters continue to render normally.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Jinja2Cpp does not check that a call target is callable and does not enforce builtin filter arity, so templates that Jinja2 rejects render an empty string.
Affected: master 79af49f (releases 1.1.0, 1.2.1 and 1.3.2).
Reproduction
{{empty()}}
Jinja2 raises TypeError: 'list' object is not callable. Jinja2Cpp renders empty output.
{{ 1|replace }}
Jinja2 raises TypeError: do_replace() missing 2 required positional arguments: 'old' and 'new'. Jinja2Cpp renders empty output.
Both templates fire on template text alone; no context variables are needed.
Expected: a render error of the same class as the Jinja2 TypeError.
Actual: no error, empty output.
Root cause
The evaluator in src/expression_evaluator.cpp dispatches calls and builtin filters without validating the callee or the argument list. In Jinja2 both are ordinary Python callables, so CPython raises before the call body runs.
- Lenguaje dominante
- C++
- Estrellas
- 602
- Forks
- 115
- 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 jinja2cpp/Jinja2Cpp
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 65/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 70/100
-
undefined reference to fmt::*Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Todos los issues de jinja2cpp/Jinja2Cpp
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
lxqt/qtermwidget#688 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
cpinitiative/usaco-guide#6665 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
mpfaffenberger/privateer_reimagined#658 ·
Los mantenedores suelen responder en 1 día
-
Broken links in the docsAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
microsoft/onnxruntime#33018 ·
Los mantenedores suelen responder en 2 días
-
Type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 2 días