apache / apache/maven-shared-jar

JarBytecodeHashAnalyzer swallows IOExceptions

Open
#150 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
4
Forks
9
Avg merge
2h 46m
Merged PRs (30d)
3

Description

### Affected version

HEAD

### Bug description

This is tricky for API compatibility, but JarBytecodeHashAnalyzer swallows IOExceptions and returns null. Instead it should let the IOException bubble up to the caller so it can handle it appropriately. Deprecate the existing method and add a new method that is declared to throw IOException.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating JarBytecodeHashAnalyzer, its existing bytecode-hash method, and the callers that depend on its current API. Review the API-compatibility implications, then verify that the existing method is deprecated, the new method declares IOException, and I/O failures reach the caller instead of returning null.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system, tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.