Servlet is invoked only for the default switch case

Question

Here when the url pattern is "/Login/new" the servlet is not invoked which I verified by printing to console.It works just for the default case that is for "/Login"

@WebServlet("/Login")
public class UserServlet extends HttpServlet {
    private static final long serialVersionUID = 1L;
       
    private UserDAO userDAO;
    
    public void init()
    {
        userDAO = new UserDAO();
    }
    
    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {

        System.out.println("Servlet invoked !!");
        String action = request.getServletPath();
        System.out.println(action);
        switch(action)
        {
        case "/new":
            showNewForm(request,response);
            break;  
        case "/insert":
            insertUser(request, response);
            break;
        case "/update":
            updateUser(request, response);
            break;
        case "/delete":
            deleteUser(request, response);
            break;
        case "/edit":
            showEditForm(request, response);
            break;
        case "/login":
            RequestDispatcher dispatcher= request.getRequestDispatcher("/login.jsp");
            dispatcher.forward(request, response);
            
            
            
        default:
            //listUser(request,response);
            showLogin(request,response);
            break;
        
        
        }

Output for "/Login":[ Output for "/Login":1

Output for "/Login/new":

The desired output is that the function in "/new" case should be called.What am I missing?

0xh3xa · Accepted Answer · 2020-07-28T11:39:55.867

1

Update the path to the following

@WebServlet("/Login/*")

, And try to check that you are retrieving the last part of the path to the URL and lower-case following

String action = request.getRequestURI();
action = action.substring(action.lastIndexOf("/")).toLowerCase();

edited Jul 28 '20 at 11:39

answered Jul 28 '20 at 11:18

0xh3xa

4,801
2
14
28

Tried this but it gives the same 404 error.The thing is the servlet is not even invoked for "/Login/new" so it never reaches the switch – Harmless Jul 28 '20 at 11:21
Okay, update the path of the servlet I have updated the solution. Hope it will work :) – 0xh3xa Jul 28 '20 at 11:25
1

Thanks now the servlet is getting invoked for "Login/new" but it still runs the default case in the switch – Harmless Jul 28 '20 at 11:32
I guess we need to check the action URL – 0xh3xa Jul 28 '20 at 11:34
The value stored in ``action`` is still ``"/Login"`` when it should be ``"/new"`` – Harmless Jul 28 '20 at 11:38
Try to use `getRequestURI()` rather than `getServletPath()` – 0xh3xa Jul 28 '20 at 11:40

Servlet is invoked only for the default switch case

1 Answers1