apache / apache/maven-scripting-plugin

UncheckedIOException

Abierto
#65 2 comentarios 0 reacciones 0 asignados Ver en GitHub
bug
Lenguaje dominante
Java
Estrellas
15
Forks
10
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

### Affected version

HEAD

### Bug description

There are several UncheckedIOExceptions for issues that can definitely happen, and that are not handled where they should be. E.g.

```
protected Object eval(ScriptEngine engine, ScriptContext context) throws ScriptException {
try (FileReader reader = new FileReader(scriptFile)) {
return engine.eval(reader, context);
} catch (IOException ex) {
throw new UncheckedIOException(scriptFile + " caused:", ex);
}
}
```

This is risky and bug prone. Ideally these should be changed to IOException, but if API compatibility prevents that it looks like most could be replaced with a ScriptException instead.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Localiza el código de evaluación de ScriptEngine que contiene el método eval y el ejemplo de FileReader/UncheckedIOException; después, revisa las demás rutas de UncheckedIOException del plugin. Comprueba si cada caso puede exponer IOException o debe mantener la compatibilidad de la API con ScriptException; se considera terminado cuando las excepciones unchecked inapropiadas se hayan reemplazado de forma coherente y el comportamiento afectado esté cubierto por la suite de pruebas existente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
build-system
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.