Mercurial > libervia-backend
changeset 758:86224a13cc1d
plugin xep-0050: using new Sessions class
author | Goffi <goffi@goffi.org> |
---|---|
date | Tue, 24 Dec 2013 15:19:08 +0100 |
parents | bbe55c7bee43 |
children | 93bd868b8fb6 |
files | src/plugins/plugin_xep_0050.py |
diffstat | 1 files changed, 6 insertions(+), 20 deletions(-) [+] |
line wrap: on
line diff
--- a/src/plugins/plugin_xep_0050.py Tue Dec 24 15:19:08 2013 +0100 +++ b/src/plugins/plugin_xep_0050.py Tue Dec 24 15:19:08 2013 +0100 @@ -24,6 +24,7 @@ from twisted.internet import defer, reactor from wokkel import disco, iwokkel, data_form from sat.core import exceptions +from sat.memory.memory import Sessions from uuid import uuid4 from zope.interface import implements @@ -82,10 +83,9 @@ self.label = label self.node = node self.features = [disco.DiscoFeature(feature) for feature in features] - self.timeout = timeout self.allowed_jids, self.allowed_groups, self.allowed_magics, self.forbidden_jids, self.forbidden_groups = allowed_jids, allowed_groups, allowed_magics, forbidden_jids, forbidden_groups self.client = client - self.sessions = {} + self.sessions = Sessions(timeout=timeout) def getName(self, xml_lang=None): return self.label @@ -155,9 +155,7 @@ self.client.xmlstream.send(result) if status in (XEP_0050.STATUS.COMPLETED, XEP_0050.STATUS.CANCELED): - timer = self.sessions[session_id][0] - timer.cancel() - self._purgeSession(session_id) + del self.sessions[session_id] def _sendError(self, error_constant, session_id, request): """ Send error stanza @@ -170,14 +168,6 @@ error_elt = iq_elt.elements(None, "error").next() error_elt.addElement(cmd_condition, NS_COMMANDS) self.client.xmlstream.send(iq_elt) - try: - timer = self.sessions[session_id][0] - timer.cancel() - self._purgeSession(session_id) - except KeyError: - pass - - def _purgeSession(self, session_id): del self.sessions[session_id] def onRequest(self, command_elt, requestor, action, session_id): @@ -185,18 +175,14 @@ return self._sendError(XEP_0050.ERROR.FORBIDDEN, session_id, command_elt.parent) if session_id: try: - timer, session_data = self.sessions[session_id] + session_data = self.sessions[session_id] except KeyError: return self._sendError(XEP_0050.ERROR.SESSION_EXPIRED, session_id, command_elt.parent) if session_data['requestor'] != requestor: return self._sendError(XEP_0050.ERROR.FORBIDDEN, session_id, command_elt.parent) - timer.reset(self.timeout) else: - # we are starting a new session - session_id = str(uuid4()) - session_data = {'requestor': requestor} - timer = reactor.callLater(self.timeout, self._purgeSession, session_id) - self.sessions[session_id] = (timer, session_data) + session_id, session_data = self.sessions.newSession() + session_data['requestor'] = requestor if action == XEP_0050.ACTION.CANCEL: d = defer.succeed((None, XEP_0050.STATUS.CANCELED, None, None)) else: