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

DefaultServlet vs ResourceServlet #12476

Open
sbernard31 opened this issue Nov 4, 2024 · 6 comments
Open

DefaultServlet vs ResourceServlet #12476

sbernard31 opened this issue Nov 4, 2024 · 6 comments
Labels

Comments

@sbernard31
Copy link

Jetty Version
12.0.14

Jetty Environment
ee10

Java Version
17

Question

I try to migrate from jetty 9 to 12.
And It seems migration is pretty smooth 👍 but I get this warning :

2024-11-04 16:05:43,996 DefaultServlet       [WARN] DefaultServlet pathInfoOnly is set to true. Use ResourceServlet instead.  
2024-11-04 16:06:09,081 DefaultServlet       [WARN] Incorrect mapping for DefaultServlet at /*. Use ResourceServlet  

Until now, I was using DefaultServlet as a way to serve static resource. (without really understand the concept)
With jetty12/ee10, I understand that I should rather use a ResourceServlet for that.

This lead me to the question what is the purpose of DefaultServlet ?
I read lot about this but still not clear to me:

I describe my use case in case it helps.
I use jetty programmatically without using xml configuration.
I serve static resource from webapp directory of the jar of my java application and I have some REST API under /api/*

You can have a look at the code jetty 9 code and or jetty 12 version one (almost the same) from eclipse-leshan/leshan#1667.

@janbartel
Copy link
Contributor

@sbernard31 have also a read of #10296. In previous versions of jetty, we had users configuring a DefaultServlet instance to serve static content other than what the DefaultServlet with pathMapping of / would normally serve. It was very complex to code to handle a last-resort mapping of /, but also another mapping of /foo/*, especially where welcome files etc were involved. To fix this headache, we split the DefaultServlet so it is now a specialization of the general ResourceServlet. The DefaultServlet should be left mapped as /. You can use the ResourceServlet instead to serve other content from a different mapping.

@sbernard31
Copy link
Author

sbernard31 commented Nov 5, 2024

@janbartel

Thx a lot for quick answer.
So I get in my case, as I want to map static resource on /, I need to use DefaultServlet.
Maybe to clarify this is should be renamed : RootResourceServlet ?

A last question to define where are the resource on the server.
Before I was using :

staticHolder.setInitParameter("resourceBase",
         LeshanServerDemo.class.getClassLoader().getResource("webapp").toExternalForm());

Now, this is deprecated and I need to use :

staticHolder.setInitParameter("baseResource",
         LeshanServerDemo.class.getClassLoader().getResource("webapp").toExternalForm());

But I also see there is a :

 ServletContextHandler context = new ServletContextHandler(ServletContextHandler.SESSIONS);
  context.setBaseResource(myResource));

What is the purpose of that API ? (What means to set a BaseResource on a context 🤔 )
Javadoc says :

Set the base resource to serve content from for this context, which must exist and be readable when the context is started.

but this is not clear to me.

@janbartel
Copy link
Contributor

@sbernard31 we have always supported ContextHandler.setBaseResource method in jetty (although several versions ago we may have changed from setResourceBase. For WebAppContexts it is automatically set from the value of the parameter to setWar. For ServletContextHandlers you have to set it yourself and this will form the root of the static resources for the DefaultServlet/ResourceServlet to serve. Alternatively, you can explicitly set it to some other value on the DefaultServlet/ResourceServlet using the init params.

@sbernard31
Copy link
Author

Ok so in my case where I have only 1 DefaultServlet in my context, this is exactly the same to do :

ServletContextHandler context = new ServletContextHandler(ServletContextHandler.SESSIONS);
context.setBaseResource(myResourceAtGivenUri));

or

staticHolder.setInitParameter("baseResource",UriAsStringOfMyResource);

@janbartel
Copy link
Contributor

@sbernard31 if this issue is now resolved, can we close it?

@sbernard31
Copy link
Author

Yep, we can close it.
Eventually, if can just confirm that my understanding at #12476 (comment) is correct 🙏 that would be great.

Thx again for your time 🙏 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants