Platform plugin output is written to standard output and breaks redirected JSON/SARIF results
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
Línea de trabajo
La redirección del flujo ocurre en ScanCmd mientras se ejecutan los plugins de plataforma, así que empieza allí y busca dónde se apunta System.out al flujo de resultados. Redirige el System.out de los plugins a System.err durante ese tramo, igual que ya se maneja System.err. Está terminado cuando bal scan --platforms=sonarqube > issues.json produce JSON válido y los logs de los plugins solo aparecen en stderr. Además, el backport a 0.11.x en #165 debe recibir el mismo cambio.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
While platform plugins run, ScanCmd routes System.out to the results stream (standard output). Anything a platform plugin prints is therefore appended to the JSON or SARIF results.
The README says only the issues are written to standard output, so bal scan > issues.json saves valid JSON. With a platform selected, it does not:
bal scan --platforms=sonarqube > issues.json
issues.json then contains the plugin's logs after the results and is no longer valid JSON.
This happens on main (0.13.0) and on the 0.11.x line (raised in the review of #165).
Suggested fix
Route the plugins' System.out to standard error while they run, the same as their System.err, so standard output holds only the results.
Affected versions
0.13.0, and the 0.11.x backport in #165.
- Lenguaje dominante
- Java
- Estrellas
- 15
- Forks
- 12
- Merge medio
- 2 d 19 h
- PR fusionados (30 d)
- 20
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin 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 ballerina-platform/static-code-analysis-tool
-
Module file paths in the JSON report are only file namesPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. AbiertoArea/Reports Type/Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ballerina-platform/static-code-analysis-tool#152 ·
Los mantenedores suelen responder en 1 día
-
Area/CLI Type/Improvement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
ballerina-platform/static-code-analysis-tool#140 ·
Los mantenedores suelen responder en 1 día
-
Area/Rules Type/Bug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
ballerina-platform/static-code-analysis-tool#179 ·
Los mantenedores suelen responder en 1 día
-
ballerina:13 reports boolean strings, auth scheme names and qualified secret namesPosiblemente ocupada @TharmiganK la tomó hace 1 día. AbiertoArea/Rules Type/Bug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
ballerina-platform/static-code-analysis-tool#178 ·
Los mantenedores suelen responder en 1 día
-
Area/Rules Type/Bug
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
ballerina-platform/static-code-analysis-tool#177 ·
Los mantenedores suelen responder en 1 día
Todos los issues de ballerina-platform/static-code-analysis-tool
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
wultra/powerauth-tests#980 ·
-
account bug communication lecture
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
line/centraldogma#1397 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
cryptomator/hub#515 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
vert-x3/vertx-mail-client#268 ·