From 2d83e10b5ab37d995c8503e8e2b631ac43cad5c7 Mon Sep 17 00:00:00 2001 From: Mike Rylander Date: Thu, 6 Nov 2014 16:03:36 -0500 Subject: [PATCH] LP#1390225: Fail to care about errors from auth.session.delete We're just tossing this call over the wall and moving on. We don't care if the session wasn't there to delete or even if we didn't pass a session to the server. Signed-off-by: Mike Rylander Signed-off-by: Chris Sharp Signed-off-by: Jason Stephenson --- Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm index 868907d22b..3d717adf3d 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm @@ -483,11 +483,13 @@ sub load_logout { # while logged in, go ahead and clear it out. $self->clear_anon_cache; - $U->simplereq( - 'open-ils.auth', - 'open-ils.auth.session.delete', - $self->cgi->cookie(COOKIE_SES) - ); + try { # a missing auth token will cause an ugly explosion + $U->simplereq( + 'open-ils.auth', + 'open-ils.auth.session.delete', + $self->cgi->cookie(COOKIE_SES) + ); + } catch Error with {}; return $self->generic_redirect( $redirect_to || $self->ctx->{home_page}, -- 2.11.0