Jest kilka problemów z kodem, który aktualnie masz.

• powinieneś użyć Interceptora, aby wymusić, że użytkownik jest zalogowany, zamiast próbować wymusić to w JSP. JSP powinien być tylko do prezentacji, a nie do kontroli przepływu.
• należy unikać skryptów (bloków kodu) w JSP. To było przestarzałe naprawdę dawno temu i jest powszechnie uważane za bardzo słabą praktykę w aplikacji MVC.
• możesz uzyskać dostęp do wartości sesji w JSP bezpośrednio. Nie musisz implementować interfejsu SessionAware w swojej akcji, chyba że potrzebujesz dostępu do sesji wewnątrz samej akcji.
• powinieneś przekierować użytkownika do akcji logowania, a nie bezpośrednio do strony JSP, w przeciwnym razie omijasz Framework Struts2 i tracisz korzyści z używania frameworku.

Przykład Logowania

Poniżej znajduje się przykładowy kod do tworzenia podstawowego systemu logowania przy użyciu frameworka Struts2.

Login Wymagane

Ta część jest opcjonalna, ale ogólnie rzecz biorąc, nie wszystkie strony w aplikacji internetowej wymagają zalogowania użytkownika. Dlatego stwórzmy interfejs o nazwie LoginRequired. Każda akcja implementująca ten interfejs znacznika przekieruje na stronę logowania, jeśli użytkownik nie jest jeszcze zalogowany.

Uwaga: Możesz zamiast tego użyć adnotacji, jeśli wolisz, ale w tym przykładzie użyję interfejsu.

public interface LoginRequired {}

Przechwytujący

Przechwytujący poradzi sobie zmuszanie użytkownika do zalogowania się dla żądanej akcji implementującej interfejs LoginRequired.

public class LoginInterceptor extends AbstractInterceptor {
    @Override
    public String intercept(final ActionInvocation invocation) throws Exception {
        Map<String, Object> session = ActionContext.getContext().getSession();

        // sb: feel free to change this to some other type of an object which
        // represents that the user is logged in. for this example, I am using
        // an integer which would probably represent a primary key that I would
        // look the user up by with Hibernate or some other mechanism.
        Integer userId = (Integer) session.get("userId");

        // sb: if the user is already signed-in, then let the request through.
        if (userId != null) {
            return invocation.invoke();
        }

        Object action = invocation.getAction();

        // sb: if the action doesn't require sign-in, then let it through.
        if (!(action instanceof LoginRequired)) {
            return invocation.invoke();
        }

        // sb: if this request does require login and the current action is
        // not the login action, then redirect the user
        if (!(action instanceof LoginAction)) {
            return "loginRedirect";
        }

        // sb: they either requested the login page or are submitting their
        // login now, let it through
        return invocation.invoke();
    }
}

Będziesz również potrzebował LoginAction, która wyświetla i przetwarza stronę logowania oraz LogoutAction, która unieważnia lub czyści sesję.

Konfiguracja

Będziesz musiał dodać interceptor do stosu, a także utworzyć globalne mapowanie wyników dla "loginRedirect".

<interceptors>
    <interceptor name="login" class="your.package.LoginInterceptor"/>

    <!-- sb: you need to configure all of your interceptors here. i'm only
         listing the one we created for this example. -->
    <interceptor-stack name="yourStack">
        ...
        <interceptor-ref name="login"/>
        ...
    </interceptor-stack>
</interceptors>

<global-results>
    <!-- sb: make this the path to your login action.
         this could also be a redirectAction type. -->
    <result name="loginRedirect" type="redirect">/login</url>
</global-results>