Вообщем, твоя проблема в том, что ты выбрал не совсем верную последовательность действий: LoginServlet не должен срабатывать первым и направлять пользователь на страницу login.jsp, где пользователь вводит данные. LoginServlet лучше вызывать тогда, когда пользователь ввел данные!
Рассмотрим следующий пример. У нас есть LoginFilter - всегда будет вызван первым (Проверяет, прошел ли пользователь аутентификацию. Если да, то направляет на домашнюю страницу index.jsp. Нет - остается на странице login.jsp), LoginServlet.
Последовательность действий:
- Пользователь делает первый запрос, поэтому не важно на какую страницу, LoginFilter перенаправить его на страницу login.jsp;
- Пользователь вводит данные и нажимает кнопку Submit;
- Снова вызывается LoginFilter (в моем примере он будет вызываться первым после каждого запроса);
- Вызывается LoginServlet, в котором происходит проверка: есть ли пользователь с такими данными в базе.
Приступим к реализации. Первым делом взглянем на web.xml
<filter>
<filter-name>LoginFilter</filter-name>
<filter-class>mypackage.filters.LoginFilter</filter-class>
</filter>
<filter-mapping>
<filter-name>LoginFilter</filter-name>
<url-pattern>/*</url-pattern>
</filter-mapping>
<servlet>
<servlet-name>LoginServlet</servlet-name>
<servlet-class>mypackage.servlets.LoginServlet</servlet-class>
</servlet>
<servlet-mapping>
<servlet-name>LoginServlet</servlet-name>
<url-pattern>/login</url-pattern>
</servlet-mapping>
Как видно url-pattern для LoginFilter - /*
. Это как раз и означает, что он будет вызван всегда. Ну и LoginServlet вызывается по url-у /login
.
public class LoginFilter implements Filter {
@Override public void init(FilterConfig config) {}
@Override public void destroy() {}
@Override public void doFilter(ServletRequest req,
ServletResponse res,
FilterChain chain) throws IOException, ServletException {
HttpServletRequest request = (HttpServletRequest) req;
HttpServletResponse response = (HttpServletResponse) res;
HttpSession session = request.getSession(false);
String loginURL = request.getContextPath() + "/login.jsp";
boolean loggedIn = session != null && session.getAttribute("user") != null;
boolean loginRequest = request.getRequestURI().equals(loginURL) || request.getRequestURI().equals(loginURL + ".xhtml"); // Эта проверка важна иначе фильтр будет вечно перенаправлять на страницу login.jsp
if(loggedIn || loginRequest) {
chain.doFilter(req, res);
}
else {
response.sendRedirect("login.jsp");
}
}
}
public class LoginServlet extends HttpServlet {
private static final Logger logger = Logger.getLogger(LoginServlet.class.getName());
@Override public void doPost(HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException {
String login = request.getParameter("login");
String password = request.getParameter("password");
Database database = new Database(); // класс который отвечает за соединение с базой данных и запросы к ней
User user = database.findUserByLogPass(login, password);
if(user != null) {
logger.log(Level.INFO, String.format("User %s has logged in", user));
request.getSession().setAttribute("user", user);
response.sendRedirect("index.jsp");
}
else {
logger.log(Level.WARNING, "User with such login and password doesnt exist");
request.getRequestDispatcher("login.jsp").forward(request, response);
}
}
}
А почему же все-таки сервлет не перенаправлял на страницу QuizAdd.jsp? Ты наткнулся на очень тонкий момент. Я действительно соврал, когда сказал что код ниже строки dispatcher.forward(req,resp);
выполнятся не будет. Дело в том, что все фильтры и сервлеты веб контейнер кладет в стек (но это не обязательно так). Принцип простой: первый вошел - последний вышел. Первым у тебя вызывается LoginServlet, который "делегирует" часть обязанностей и вызывает другой сервлет (который веб контейнер создает на основе страницы .jsp
). После того, как этот сервлет выполнил свою работу, продолжает выполняться LoginServlet (это, то почему ты видел сообщения). Но проблема в том, что сервлет (на основе страницы .jsp
) уже отправил пользователю ответ (поток OutputStream уже закрыт) и поэтому никакого перенаправления не происходит (ответ уже отправлен). Правда странно, что ты не получил ошибку IllegalStateException
(она появляется как раз тогда, когда пытаешься дописать что-то в OutputStream когда он закрыт).
Надеюсь мой ответ тебе помог :)
resp.getWriter().print("Invalid credentions")
?dispatcher.forward(req,resp);
. Вы сразу перенаправляете запрос странице Authorization.jsp (а точнее сервлету, который будет создан веб контейнером). Собственно, Вы получаете страницу Authorization.jsp и все что ниже данной строкиdispatcher.forward(req,resp);
просто не выполняется.