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.

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

Starting Jetty with JPMS produces warnings about Servlet resources not found #5901

Closed
sbordet opened this issue Jan 21, 2021 · 1 comment
Closed
Assignees

Comments

@sbordet
Copy link
Contributor

sbordet commented Jan 21, 2021

Jetty version
10.0.x & 11.0.x

Description
Start Jetty with --jpms and you'll see a number of warning like the one below:

2021-01-21 16:18:10.049:WARN :oatud.DigesterFactory:main: The XML schema [XMLSchema.dtd] could not be found. This is very likely to break XML validation if XML validation is enabled.
@janbartel janbartel self-assigned this Jan 25, 2021
janbartel added a commit that referenced this issue Jan 26, 2021
This brings in a change to the javax.servlet.api module-info
to open the javax.servlet.resources package to all modules,
so that the dtds and xsds can be retrieved as resources, either
from the context Classloader.getResource, or
from the ServletContext.class.getResource.

Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Jan 26, 2021
Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Jan 27, 2021
This brings in a change to the javax.servlet.api module-info
to open the javax.servlet.resources package to all modules,
so that the dtds and xsds can be retrieved as resources, either
from the context Classloader.getResource, or
from the ServletContext.class.getResource.

Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Jan 27, 2021
Signed-off-by: Jan Bartel <janb@webtide.com>
@janbartel
Copy link
Contributor

Fixed by releases 4.0.6 and 5.0.2 of the toolchain servlet-api, see pr #5918 and #5919.

This was referenced Mar 6, 2021
This was referenced Mar 10, 2021
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

No branches or pull requests

2 participants