Skip to content

[Backport 4.4.x] Filter out empty repository entries in feature descriptors#2520

Merged
jbonofre merged 1 commit intoapache:karaf-4.4.xfrom
jbonofre:backport/filter-empty-repository-specs-4.4.x
Apr 16, 2026
Merged

[Backport 4.4.x] Filter out empty repository entries in feature descriptors#2520
jbonofre merged 1 commit intoapache:karaf-4.4.xfrom
jbonofre:backport/filter-empty-repository-specs-4.4.x

Conversation

@jbonofre
Copy link
Copy Markdown
Member

@jbonofre jbonofre commented Apr 5, 2026

Backport of #2515 to karaf-4.4.x branch.

  • Empty or whitespace-only <repository> elements in feature XML descriptors cause a RuntimeException from pax-url-mvn (NullArgumentException: Repository spec is empty string) during boot feature installation.
  • Features.trim() now removes empty strings instead of keeping them in the list.
  • RepositoryImpl.getRepositories() and getResourceRepositories() now filter out blank entries before calling URI.create().

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 5, 2026

Test Results

242 files  ±0  242 suites  ±0   27m 58s ⏱️ +37s
994 tests +1  951 ✅ +1  43 💤 ±0  0 ❌ ±0 
998 runs  +1  955 ✅ +1  43 💤 ±0  0 ❌ ±0 

Results for commit 8abe170. ± Comparison against base commit 1c806ee.

Empty or whitespace-only <repository> elements in feature XML
descriptors cause a RuntimeException from pax-url-mvn
(NullArgumentException: Repository spec is empty string) during
boot feature installation.

Both Features.trim() and RepositoryImpl.getRepositories()/
getResourceRepositories() now discard blank entries instead of
passing them through to URI creation.

(backport of apache#2515)
@jbonofre jbonofre force-pushed the backport/filter-empty-repository-specs-4.4.x branch from 8abe170 to cba76fe Compare April 15, 2026 20:24
@jbonofre jbonofre merged commit 73cd9ee into apache:karaf-4.4.x Apr 16, 2026
6 checks passed
@jbonofre jbonofre deleted the backport/filter-empty-repository-specs-4.4.x branch April 16, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant