From a7530dc33779ae894e4f549006443a247aae50c6 Mon Sep 17 00:00:00 2001 From: Madhur Garg Date: Thu, 25 Jul 2019 02:00:15 +0530 Subject: Added a check for IqError and IqTimeout exceptions, while doing mam and disco query. --- poezio/mam.py | 34 +++++++++++++++++++++++++--------- 1 file changed, 25 insertions(+), 9 deletions(-) diff --git a/poezio/mam.py b/poezio/mam.py index 29efc8ff..effad4c2 100644 --- a/poezio/mam.py +++ b/poezio/mam.py @@ -8,6 +8,7 @@ import asyncio from datetime import datetime, timedelta, timezone +from slixmpp.exceptions import IqError, IqTimeout from poezio.theming import get_theme from poezio import tabs from poezio.text_buffer import Message, TextBuffer @@ -46,23 +47,38 @@ async def query(self, remote_jid, start, end, top): self.start_date = start self.end_date = end text_buffer = self._text_buffer - iq = await self.core.xmpp.plugin['xep_0030'].get_info(jid=remote_jid) + try: + iq = await self.core.xmpp.plugin['xep_0030'].get_info(jid=remote_jid) + except (IqError, IqTimeout): + return self.information('Failed to retrieve messages', 'Error') if 'urn:xmpp:mam:2' not in iq['disco_info'].get_features(): return self.core.information("This MUC doesn't support MAM.", "Error") if top: if isinstance(self, tabs.MucTab): - results = self.core.xmpp['xep_0313'].retrieve(jid=self.remote_jid, - iterator=True, reverse=top, end=self.end_date) + try: + results = self.core.xmpp['xep_0313'].retrieve(jid=self.remote_jid, + iterator=True, reverse=top, end=self.end_date) + except (IqError, IqTimeout): + return self.information('Failed to retrieve messages', 'Error') else: - results = self.core.xmpp['xep_0313'].retrieve(with_jid=self.remote_jid, - iterator=True, reverse=top, end=self.end_date) + try: + results = self.core.xmpp['xep_0313'].retrieve(with_jid=self.remote_jid, + iterator=True, reverse=top, end=self.end_date) + except (IqError, IqTimeout): + return self.information('Failed to retrieve messages', 'Error') else: if isinstance(self, tabs.MucTab): - results = self.core.xmpp['xep_0313'].retrieve(jid=self.remote_jid, - iterator=True, reverse=top, start=self.start_date, end=self.end_date) + try: + results = self.core.xmpp['xep_0313'].retrieve(jid=self.remote_jid, + iterator=True, reverse=top, start=self.start_date, end=self.end_date) + except (IqError, IqTimeout): + return self.information('Failed to retrieve messages', 'Error') else: - results = self.core.xmpp['xep_0313'].retrieve(with_jid=self.remote_jid, - iterator=True, reverse=top, start=self.start_date, end=self.end_date) + try: + results = self.core.xmpp['xep_0313'].retrieve(with_jid=self.remote_jid, + iterator=True, reverse=top, start=self.start_date, end=self.end_date) + except (IqError, IqTimeout): + return self.information('Failed to retrieve messages', 'Error') msg_count = 0 msgs = [] timestamp = datetime.now() -- cgit v1.2.3