IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2025-02-25 18:55:28 -06:00
Code Issues Packages Projects Releases Wiki Activity

Change session logout to kill only the cookie

Browse Source 
Removing the ccache goes too far as it will cause unrelated sessions to
fail as well, this is a problem for accounts used to do unattended
operations and that may operate in parallel.

Fixes https://fedorahosted.org/freeipa/ticket/6682

Signed-off-by: Simo Sorce <simo@redhat.com>
Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
This commit is contained in:
Simo Sorce 2017-02-16 11:07:31 -05:00 committed by Martin Basti
parent 08b8bfa9b5
commit b895f4a34b
2 changed files with 3 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
ipaserver/plugins/session.py
View File

@ -5,7 +5,6 @@
from ipalib import Command from ipalib import Command
from ipalib.request import context from ipalib.request import context
from ipalib.plugable import Registry from ipalib.plugable import Registry
from ipaserver.session import logout
register = Registry() register = Registry()
@ -21,7 +20,9 @@ class session_logout(Command):
ccache_name = getattr(context, 'ccache_name', None) ccache_name = getattr(context, 'ccache_name', None)
if ccache_name is None: if ccache_name is None:
self.debug('session logout command: no ccache_name found') self.debug('session logout command: no ccache_name found')
else:
delattr(context, 'ccache_name')
logout(ccache_name) setattr(context, 'logout_cookie', '')
return dict(result=None) return dict(result=None)

34
ipaserver/session.py
View File

@ -1,34 +0,0 @@
# Authors: John Dennis <jdennis@redhat.com>
#
# Copyright (C) 2011 Red Hat
# see file 'COPYING' for use and warranty information
#
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation, either version 3 of the License, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program. If not, see <http://www.gnu.org/licenses/>.
import os
from ipalib.request import context
from ipalib.krb_utils import (
krb5_parse_ccache,
)
def logout(ccache_name=None):
if ccache_name is None:
ccache_name = getattr(context, 'ccache_name', None)
if ccache_name is not None:
scheme, name = krb5_parse_ccache(ccache_name)
if scheme == 'FILE':
os.unlink(name)
setattr(context, 'logout_cookie', '')