Trailing comma in unpackTypes causes empty string element and confusing error
Maintainers usually reply within 1 day
Assessment
This issue has not been assessed yet.
Description
Description
In EarMojo.java:460, a trailing comma in the unpackTypes parameter produces an empty string element:
unpackTypesList = Arrays.asList(unpackTypes.split(","));
If the user configures unpackTypes = "jar,war,", split(",") produces the array ["jar", "war", ""]. The empty string "" will fail isStandardArtifactType() and produce a confusing error message.
Expected behavior
Filter out empty strings:
- Dominant language
- Java
- Stars
- 13
- Forks
- 30
- Avg merge
- 1d 3h
- Merged PRs (30d)
- 11
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/maven-ear-plugin
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 90/100
apache/maven-ear-plugin#519 · 1 comment ·
Maintainers usually reply within 1 day
-
Getter with side-effect mutation in AbstractEarModule.getBundleDir() and getLibDir()Possibly taken @efegokdemir claimed this 17 days ago. Openbug
Difficulty 1/5 Under an hour Newbie friendliness 86/100
apache/maven-ear-plugin#516 ·
Maintainers usually reply within 1 day
-
Return value of File.delete() ignored in EarMojoPossibly taken @efegokdemir claimed this 17 days ago. Openbug
Difficulty 1/5 Under an hour Newbie friendliness 85/100
apache/maven-ear-plugin#517 ·
Maintainers usually reply within 1 day
-
InterpolationException silently swallowed in AbstractEarModule.getBundleFileName()Possibly taken @efegokdemir claimed this 17 days ago. Openbug
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
apache/maven-ear-plugin#515 ·
Maintainers usually reply within 1 day
-
NullPointerException in AbstractEarModule.resolveArtifact() if earExecutionContext not setPossibly taken @efegokdemir claimed this 17 days ago. Openbug
Difficulty 1/5 Under an hour Newbie friendliness 86/100
apache/maven-ear-plugin#514 ·
Maintainers usually reply within 1 day
All issues in apache/maven-ear-plugin
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
utopia-rise/godot-jvm#1004 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
resilience4j/resilience4j#2547 ·
Maintainers usually reply within 9 days
-
Clock.MakeDate continues execution and returns a rolled-over instant after dispatching error on invalid datePossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 1/5 Under an hour Newbie friendliness 82/100
mit-cml/appinventor-sources#4155 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 1-3 hours Newbie friendliness 62/100
Hira-shi/PW1-DAI-Carrel-Egal-Eyer#28 ·
Maintainers usually reply within 1 day
-
`GET /v1/event/token/{uuid}` can report a BOM upload as done before policy evaluation and metrics have finishedPossibly taken @Zargath claimed this today. Opendefect in triage
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
DependencyTrack/dependency-track#7646 ·
Maintainers usually reply within 1 day