Mercurial > libervia-pubsub
changeset 435:96342e7e9f5d
privilege: log error when `requestInfo` is failing
| author | Goffi <goffi@goffi.org> |
|---|---|
| date | Thu, 04 Feb 2021 09:54:53 +0100 |
| parents | f6a225476165 |
| children | 80aca1cf1543 |
| files | sat_pubsub/privilege.py |
| diffstat | 1 files changed, 14 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/sat_pubsub/privilege.py Thu Feb 04 09:38:17 2021 +0100 +++ b/sat_pubsub/privilege.py Thu Feb 04 09:54:53 2021 +0100 @@ -30,7 +30,7 @@ from twisted.python import failure from twisted.internet import defer from twisted.words.xish import domish -from twisted.words.protocols.jabber import jid +from twisted.words.protocols.jabber import jid, error import time FORWARDED_NS = 'urn:xmpp:forward:0' @@ -258,10 +258,19 @@ if disco_tuple not in self.hash_map: # first time we se this hash, what is behind it? - infos = yield self.requestInfo(from_jid) - self.hash_map[disco_tuple] = { - 'notify': {f[:-7] for f in infos.features if f.endswith('+notify')}, - 'infos': infos + try: + infos = yield self.requestInfo(from_jid) + except error.StanzaError as e: + log.msg( + f"WARNING: can't request disco info for {from_jid!r} (presence: " + f"{presence_type}): {e}" + ) + else: + self.hash_map[disco_tuple] = { + 'notify': { + f[:-7] for f in infos.features if f.endswith('+notify') + }, + 'infos': infos } # jid_caps must be filled only after hash_map is set, to be sure that