TPAC logout does not delete session

Bug #1248636 reported by Jeff Godin
256
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
High
Unassigned
2.5
Fix Released
High
Unassigned
2.6
Fix Released
High
Unassigned

Bug Description

When a user logs out of the TPAC public catalog interface, the session is not deleted. The cookie on the client is cleared, but the session remains in memcached, and the authentication token can continue to be used, if it was retained via some means other than the standard browser cookie mechanisms.

The session will eventually expire after a configurable period of inactivity. The period will vary depending on if the session was created as a standard or as a persistent session.

In the JSPAC and Staff Client, "logout" deletes the session using a call to open-ils.auth.session.delete

We should do the same thing in TPAC.

Ben Shum (bshum)
Changed in evergreen:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Mike Rylander (mrylander) wrote :

There's a branch for that in the security repo called: user/miker/lp1248636_logout-delete-session

tags: added: pullrequest
Revision history for this message
Galen Charlton (gmc) wrote :

I've tested and pushed a signoff to the branch user/gmcharlt/lp1248636_signoff in the security repo.

tags: added: signedoff
Changed in evergreen:
status: Triaged → Confirmed
Revision history for this message
Jason Stephenson (jstephenson) wrote :

FWIW, I tested the branch today and confirmed that it works. I also cherry-picked it into local copies of rel_2_5 and rel_2_6 just to verify that it backports cleanly, whcih it does. (No surprise there, but just making sure.)

I won't bother signing off since Galen already has.

Galen Charlton (gmc)
Changed in evergreen:
milestone: none → 2.7.1
status: Confirmed → Fix Released
information type: Private Security → Public Security
To post a comment you must log in.
This report contains Public Security information  
Everyone can see this security related information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.