Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ServletContext.getResourceAsStream(String) must use URLConnection per Servlet Spec. #11536

Open
joakime opened this issue Mar 20, 2024 · 1 comment · May be fixed by #11537
Open

ServletContext.getResourceAsStream(String) must use URLConnection per Servlet Spec. #11536

joakime opened this issue Mar 20, 2024 · 1 comment · May be fixed by #11537
Assignees
@joakime
Labels
Specification For all industry Specifications (IETF / Servlet / etc)

Comments

@joakime
Copy link
Contributor

joakime commented Mar 20, 2024

Jetty version(s)
12.0.7

Jetty Environment
Any

Java version/vendor (use: java -version)
Any

OS type/version
Any

Description
Per the Servlet Spec API docs on ServletContext.getResourcesAsStream(String) the returned InputStream must use URLConnection.
The implementation of ServletContext in ee10 / ee9 / ee8 does not use URLConnection.
@joakime joakime added Bug For general bugs on Jetty side Specification For all industry Specifications (IETF / Servlet / etc) labels Mar 20, 2024
@joakime joakime self-assigned this Mar 20, 2024
joakime added a commit that referenced this issue Mar 20, 2024
@joakime
Issue #11536 - ServletContext.getResourceAsStream(String) must use UR…
fdb0d5b 
…LConnection
@joakime joakime linked a pull request Mar 20, 2024 that will close this issue
Open
@janbartel janbartel removed the Bug For general bugs on Jetty side label Mar 25, 2024
@janbartel
Copy link
Contributor

I'm not convinced this is a bug, so I've removed that label. We can discuss before proceeding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joakime joakime

Labels
Specification For all industry Specifications (IETF / Servlet / etc)
Projects
Jetty 12.0.10
Status: No status
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #11536 - ServletContext.getResourceAsStream(String) must use URLConnection jetty/jetty.project
2 participants
@joakime @janbartel