Review ResourceFactory.newSystemResource(String) behavior & javadoc
#10328
Assignees
Labels
Comments
joakime
added a commit
that referenced
this issue
Sep 15, 2023
+ Create a new ResourceFactory.newClassLoaderResource(String, boolean) + Make .newSystemResource(String) use it + Make .newClassPathResource(String) use it + Deprecate .newSystemResource(String) + Deprecate .newClassPathResource(String) + Adjust own codebase to not use deprecated methods
joakime
added a commit
that referenced
this issue
Sep 20, 2023
* Issue #10328 - Review ResourceFactory.newSystemResource + Create a new ResourceFactory.newClassLoaderResource(String, boolean) + Make .newSystemResource(String) use it + Make .newClassPathResource(String) use it + Deprecate .newSystemResource(String) + Deprecate .newClassPathResource(String) + Adjust own codebase to not use deprecated methods
|
Merged PR #10533 |
github-project-automation
bot
moved this from 🏗 In progress
to ✅ Done
in Jetty 12.0.2 FROZEN
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Jetty version(s)
12.0.0
Enhancement Description
The
ResourceFactory.newSystemResource(String)has a strange implementation, an awkward name, and poor javadoc attempting to explain it. (which seems at odds with each other).We should review that implementation and attempt to clean it up to be more sane.
The text was updated successfully, but these errors were encountered: