Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-40867

Timed Assessment thread should perform logout in finally to prevent a session leak

    Details

    • 19 status:
      Resolved
    • Test Plan:
      Hide

      Please add a Test Plan here.

      Show
      Please add a Test Plan here.

      Description

      Since this Thread is managed by the ScheduledExecutorService which maintains a pool of threads, meaning threads are reused, we need to be diligent about cleaning up any state that may have been injected into the threads ThreadLocal. One thing to remember is that these threads contain user sessions so extra care is needed.

      Also the session logout should be moved to a finally to always ensure that happens.

      Additionally we remove the use of covers and cleanup logging.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  ern Earle R Nietzel
                  Reporter:
                  ern Earle R Nietzel
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Git Source Code