7
votes

Context

I work on a web-app (using Play Framework) and I'm trying to migrate to traditional Servlet model with Spring MVC. I'd like to run in an embedded Jetty container alongs an already existing one (netty).

Problem

I'm trying to re-use a created Spring context (that contains all application beans, including newly-added Spring MVC controllers), but the request mappings aren't picked up.

I debugged Spring's Dispatcher Servlet and there were no mappings registered indeed (so it could handle no path).

Attempted solution

Here's the manual Jetty setup code:

@RequiredArgsConstructor
public class EmbeddedJetty {

    private final int port;
    private final AnnotationConfigWebApplicationContext existingContext;

    @SneakyThrows
    public void start() {
        Assert.notNull(existingContext.getBean(UserController.class));

        val server = new Server(port);
        ServletContextHandler handler = new ServletContextHandler();
        ServletHolder servlet = new ServletHolder(new DispatcherServlet(existingContext));
        handler.addServlet(servlet, "/");
        handler.addEventListener(new ContextLoaderListener(existingContext));
        server.setHandler(handler);

        server.start();
        log.info("Server started at port {}", port);
    }

}

And here's the controller being ignored:

@Controller
public class UserController {

    @GetMapping("/users/{userId}")
    public ResponseEntity<?> getUser(@PathVariable("userId") long userId) {
        return ResponseEntity.ok("I work");
    }

}

Question

What do I needed to do to make my embedded jetty setup pick up the existing controller beans and serve the mappings?

4
I am actually at a loss here, I feed directly the webApplicationContext to the call starting my jetty, and the jetty starts with the mapping and everything correctly. (see my edit) My last question would be what are the version of the components you use?Adonis
Do you happen to have that code on Github or somewhere? I'm on Spring 4.3.6 and Jetty 9.4.6Xorty
Actually it is on my work computer, so I had to recreate it from almost scratch (plus what's here), let me know what differs from yours: github.com/asettouf/SpringMVCTemplateAdonis
So did you find a way to solve your issue? The only thing I can think of is that you start the Jetty with a not fully initialized SpringContext, and so we would need more details regarding this part to help youAdonis
Sorry it took me so long, I posted the answer.Xorty

4 Answers

2
votes

I believe you are missing the MVC Java Config that handles request to @RequestMapping inside your Controller.

So basically what you would need to add is a WebMVC config class like:

package my.spring.config;
//necessary imported packages avoided for shortening the example
@EnableWebMvc 
@Configuration
@ComponentScan({ "my.jetty.test" })
public class SpringWebConfig extends WebMvcConfigurerAdapter {

}

Then you would need to indicate to your AnnotationConfigWebApplicationContextwhere the config is located, add to your startmethod this:

webAppContext.setConfigLocation("my.spring.config");

And voila, the below (very simple) Controller serves request on my localhost:

package my.jetty.test;
//necessary imported packages avoided for shortening the example
@Controller
public class HelloController {

    @GetMapping(value = "/")
    @ResponseBody
    public String printWelcome(HttpServletRequest request) {
        return "hello";
    }

    @GetMapping(value = "/hello/{name:.+}")
    @ResponseBody
    public String hello(@PathVariable("name") String name) {
        return "hello " + name;
    }
}

If needed, I can give the full example. Several links that helped me:

Edit: The repo with my code working

1
votes

Static Structure

If you are going to migrate to servlet model, you may would like to get familiar with the normal structure:

    .
    ├── pom.xml
    ├── README
    ├── src
    │   ├── main
    │   │   ├── java
    │   │   ├── resources
    │   │   │   ├── spring
    │   │   └── webapp
    │   │       └── WEB-INF
    │   │           └── web.xml
    │   └── test
    │       ├── java
    │       └── resources

The web.xml is the core descriptor that j2ee server used to deploy the app. There exists some important components in a app which is defined in web.xml:

  • filter
  • servlet
  • listener

Sever Start

When server starts, it will setup listener for monitoring the lifecycle of whole app or a request; it will setup filter to filter requests; it will setup servlet to handle request.

Spring Way

Spring is a very light-weight way to integrate many convenient method/utility into our applications. It is light weight because it is attached into our projects only by two things:

  • Define spring's listener in web.xml to initialize
  • Direct all requests to spring (for Spring MVC)

Suggestions

So, back to our problem.

  • Your jetty server start with a ServletContextHandler which is just related to mapping, but not listener (i.e. no spring config will be init). You should start with WebAppContext;
  • You at least should add web.xml to pick up the existing controller beans and serve the mappings;

Ref

1
votes

This ended up being a bit of pain and re-work, but the final solution was making sure that existingContext is NOT started and refreshed prior to starting the DispatcherServlet.

-1
votes

Try this in your Web configuration file

@ComponentScan(basePackages = {"com.our.controller"})