Skip to content

fix: support Commons IO 2.22 BOM streams - #4864

Merged
tenthe merged 1 commit into
apache:dependabot/maven/commons-io-commons-io-2.22.0from
skrcode:fix-commons-io-2-22-bom-stream
Sep 3, 2026
Merged

fix: support Commons IO 2.22 BOM streams#4864
tenthe merged 1 commit into
apache:dependabot/maven/commons-io-commons-io-2.22.0from
skrcode:fix-commons-io-2-22-bom-stream

Conversation

@skrcode

@skrcode skrcode commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Purpose

Completes the Commons IO 2.22 upgrade from #4625.

Commons IO 2.22 makes BOMInputStream construction report IOException. This replaces the deprecated constructor with the builder API and propagates that checked failure through the existing file-storage boundary, which already reports IOException.

Remarks

  • Exact source revision: 9662f633d73660353bd48317e7b7ffb39769d3d7
  • Scope: one production file; no dependency or unrelated changes
  • Verification: mvn -pl streampipes-pipeline-management -am test on JDK 25 / Maven 3.9.16
    • all 25 reactor modules succeeded
    • 95 tests, 0 failures or errors
    • Checkstyle and dependency convergence passed
  • The build itself completed successfully; only the subsequent Develocity scan publication could not reach its server.

PR introduces breaking changes: no

PR introduces deprecations: no

AI assistance

Built and verified with JAIPilot:

  • Skills: jaipilot-maintainer-intent, jaipilot-fast-execution, jaipilot-review-diff, and jaipilot-remote-java
  • Execution profile: gpt-5.6-sol · xhigh · fast

Signed-off-by: Suraj Rajan <surajkrajan95@gmail.com>
@tenthe
tenthe merged commit 9252aae into apache:dependabot/maven/commons-io-commons-io-2.22.0 Sep 3, 2026
3 checks passed
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.

2 participants