From 8e388df8e0d2f6443a0a5deab59ecc355b82532b Mon Sep 17 00:00:00 2001
From: mathieui <mathieui@mathieui.net>
Date: Fri, 26 Feb 2021 00:07:08 +0100
Subject: XEP-0231: do not crash if max-age is None

it is only RECOMMENDED
---
 slixmpp/plugins/xep_0231/bob.py    | 3 ++-
 slixmpp/plugins/xep_0231/stanza.py | 8 ++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

(limited to 'slixmpp/plugins/xep_0231')

diff --git a/slixmpp/plugins/xep_0231/bob.py b/slixmpp/plugins/xep_0231/bob.py
index 129aca56..30722208 100644
--- a/slixmpp/plugins/xep_0231/bob.py
+++ b/slixmpp/plugins/xep_0231/bob.py
@@ -89,7 +89,8 @@ class XEP_0231(BasePlugin):
         bob['data'] = data
         bob['type'] = mtype
         bob['cid'] = cid
-        bob['max_age'] = max_age
+        if max_age is not None:
+            bob['max_age'] = max_age
 
         await self.api['set_bob'](args=bob)
         # Schedule destruction of the data
diff --git a/slixmpp/plugins/xep_0231/stanza.py b/slixmpp/plugins/xep_0231/stanza.py
index 809253d4..6bde671b 100644
--- a/slixmpp/plugins/xep_0231/stanza.py
+++ b/slixmpp/plugins/xep_0231/stanza.py
@@ -18,10 +18,14 @@ class BitsOfBinary(ElementBase):
     interfaces = {'cid', 'max_age', 'type', 'data'}
 
     def get_max_age(self):
-        return int(self._get_attr('max-age'))
+        try:
+            return int(self._get_attr('max-age'))
+        except ValueError:
+            return None
 
     def set_max_age(self, value):
-        self._set_attr('max-age', str(value))
+        if value is not None:
+            self._set_attr('max-age', str(value))
 
     def get_data(self):
         return base64.b64decode(bytes(self.xml.text))
-- 
cgit v1.2.3