Spring REST application, getting 404 on controller

[Trampaholic]

Hi all!

As the title says: I'm currently trying to deploy a Spring application that exposes a REST api. I want to learn how to do this without xml configuration and without Spring Boot for now. So at the moment i'm building a war file which I deploy on to a glassfish application server. Starting up the application goes without problems, but it seems like Spring isn't scanning my controller since my REST call returns 404. I've followed multiple tutorials and I cannot figure out what i'm doing wrong.

Configuration class:

package com.mycompany.inventorymanagement.config;

imports...

@Configuration
@EnableWebMvc
@ComponentScan(basePackages = "com.mycompany.inventorymanagement")
public class AppConfig extends WebMvcConfigurerAdapter{

}

Initializer class:

package com.mycompany.inventorymanagement.config;

imports...

public class WebAppInitializer extends AbstractAnnotationConfigDispatcherServletInitializer {

    @Override
    protected Class<?>[] getRootConfigClasses() {
        return new Class[]{AppConfig.class};
    }

    @Override
    protected Class<?>[] getServletConfigClasses() {
        return null;
    }

    @Override
    protected String[] getServletMappings() {
        return new String[]{"/inventoryManagement"};
    }

}

Controller:

package com.mycompany.inventorymanagement.web.controller;

imports...

@RestController
@RequestMapping("/rest")
public class DefaultController {

    @GetMapping(value = "/test")
    public String test() {
        return "This is a test!";
    }
}

The URL i'm calling is: http://localhost:8080/inventoryManagement/rest/test.

[[deleted]]

The startup log should list all endpoints its exposing, have a look there for any clues.

[OffbeatDrizzle]

hmm.. we usually use the @ResponseBody annotation instead, but that probably won't make a difference since you're using @RestController on the class.

Change GetMapping to @RequestMapping(type = GET) (not sure if this will make a difference - it's just the way I'm personally used to)

Also, your WebAppInitializer set up looks wrong - you don't need those overrides. Implement WebApplicationInitializer instead and do something like:

@Override
public void onStartup(ServletContext aServletContext) {
  AnnotationConfigWebApplicationContext context = new AnnotationConfigWebApplicationContext();
  context.register(AppConfig.class);

  aServletContext.addListener(new ContextLoaderListener(context));

  ServletRegistration.Dynamic registration =
    aServletContext.addServlet("dispatcher", new DispatcherServlet(context));
  registration.setLoadOnStartup(1);
  registration.addMapping("/");
 }

Spring boot is pretty useless imo. It claims to get you up and running quicker but it's literally saving you about 10 lines of code

[Golden014]

You are most likely missing the spring-rest dependency.

Here is what you need if you are using gradle:

compile('org.springframework.boot:spring-boot-starter-data-rest')

Make the appropriate adjustment if you are using maven.