Wicket
  1. Wicket
  2. WICKET-5103

Wicket session id not up to date when container changes session id

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.6.0
    • Fix Version/s: 6.8.0, 7.0.0-M1
    • Component/s: wicket
    • Labels:
      None

      Description

      When the web container changes the session id, Wicket session still holds previous session id and never updates.

      The attached quickstart uses basic form authentication, so Jetty will change the session id after login.

        Issue Links

          Activity

          Sven Meier created issue -
          Sven Meier made changes -
          Field Original Value New Value
          Link This issue relates to WICKET-4512 [ WICKET-4512 ]
          Hide
          Sven Meier added a comment -

          quickstart

          Show
          Sven Meier added a comment - quickstart
          Sven Meier made changes -
          Attachment WICKET-5103.zip [ 12573999 ]
          Sven Meier made changes -
          Description When the web container changes the session id, Wicket session still holds previous session id and never updates. When the web container changes the session id, Wicket session still holds previous session id and never updates.

          The attached quickstart uses basic form authentication, so Jetty will change the session id after login.
          Hide
          Sven Meier added a comment -

          The session id is now updated in #detach() as in Wicket 1.4.x

          Show
          Sven Meier added a comment - The session id is now updated in #detach() as in Wicket 1.4.x
          Sven Meier made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 6.7.0 [ 12323964 ]
          Fix Version/s 1.5.11 [ 12324069 ]
          Resolution Fixed [ 1 ]
          Sven Meier made changes -
          Affects Version/s 1.5.10 [ 12323510 ]
          Sven Meier made changes -
          Fix Version/s 1.5.11 [ 12324069 ]
          Sven Meier made changes -
          Link This issue is related to WICKET-5164 [ WICKET-5164 ]
          Hide
          Sven Meier added a comment -

          I reverted the unreleased change for 1.5.x, since this issue is more complicated (see WICKET-5164).

          Show
          Sven Meier added a comment - I reverted the unreleased change for 1.5.x, since this issue is more complicated (see WICKET-5164 ).
          Hide
          Sven Meier added a comment -

          PageStoreManager.SessionEntry caches sessionId too

          Show
          Sven Meier added a comment - PageStoreManager.SessionEntry caches sessionId too
          Sven Meier made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Hide
          Sven Meier added a comment -

          PageStoreManager now uses the same sessionId to store and get pages to/from the IPageStore.

          Keeping SessionEntry#sessionId in-sync with the actual container session id will be tracked in WICKET-5164.

          Show
          Sven Meier added a comment - PageStoreManager now uses the same sessionId to store and get pages to/from the IPageStore. Keeping SessionEntry#sessionId in-sync with the actual container session id will be tracked in WICKET-5164 .
          Sven Meier made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Fix Version/s 6.8.0 [ 12324068 ]
          Fix Version/s 6.7.0 [ 12323964 ]
          Resolution Fixed [ 1 ]
          Hide
          Martin Grigorov added a comment -

          I see only commit in master branch. This means it wont be included in 6.8.0. You have to port to wicket-6.x for 6.8.0.

          Show
          Martin Grigorov added a comment - I see only commit in master branch. This means it wont be included in 6.8.0. You have to port to wicket-6.x for 6.8.0.
          Sven Meier made changes -
          Fix Version/s 7.0 [ 12322958 ]
          Hide
          Sven Meier added a comment -

          Thanks!

          Show
          Sven Meier added a comment - Thanks!
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          4d 8h 11m 1 Sven Meier 20/Mar/13 15:56
          Resolved Resolved Reopened Reopened
          40d 20h 19m 1 Sven Meier 30/Apr/13 13:15
          Reopened Reopened Resolved Resolved
          5m 10s 1 Sven Meier 30/Apr/13 13:20

            People

            • Assignee:
              Sven Meier
              Reporter:
              Sven Meier
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development