apache / apache/maven-scripting-plugin

UncheckedIOException

Offen
#65 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug
Vorherrschende Sprache
Java
Sterne
15
Forks
10
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

### 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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Finde den Evaluierungscode von ScriptEngine, der die eval-Methode und das FileReader/UncheckedIOException-Beispiel enthält, und überprüfe anschließend die anderen UncheckedIOException-Pfade im Plugin. Prüfe, ob jeder Fall IOException offenlegen kann oder die API-Kompatibilität mit ScriptException wahren muss; abgeschlossen ist die Aufgabe, wenn die unangemessenen unchecked Exceptions konsistent ersetzt wurden und das betroffene Verhalten durch die bestehende Testsuite abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
build-system
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.