Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-2151

WicketSessionFilter doesn't takes into account WebApplication#getSessionAttributePrefix(WebRequest)

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.3.5
    • 1.3.6, 1.4-RC3
    • wicket
    • None
    • Any

    Description

      WicketSessionFilter#init(FilterConfig), line 139 constructs the 'sessionKey' without taking into account the return value of WebApplication#getSessionAttributePrefix(WebRequest).

      Patch:

      Index: protocol/http/servlet/WicketSessionFilter.java
      ===================================================================
      — protocol/http/servlet/WicketSessionFilter.java (revisiĆ³n: 725053)
      +++ protocol/http/servlet/WicketSessionFilter.java (copia de trabajo)
      @@ -28,6 +28,7 @@
      import javax.servlet.http.HttpSession;

      import org.apache.wicket.Session;
      +import org.apache.wicket.protocol.http.WebApplication;
      import org.slf4j.Logger;
      import org.slf4j.LoggerFactory;

      @@ -136,7 +137,8 @@
      log.debug("filterName/ application key set to " + filterName);
      }

      • sessionKey = "wicket:" + filterName + ":" + Session.SESSION_ATTRIBUTE_NAME;
        + WebApplication application = WebApplication.get(filterName);
        + sessionKey = application.getSessionAttributePrefix(null) + Session.SESSION_ATTRIBUTE_NAME;

      if (log.isDebugEnabled())
      {

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ivaynberg Igor Vaynberg
            martin-g Martin Grigorov (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment