Don't unnecessarily refetch the sender twice

LonamiWebs · Oct 3, 2022 · db29e9b · db29e9b
1 parent 299b090
commit db29e9b
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/telethon/tl/custom/sendergetter.py b/telethon/tl/custom/sendergetter.py
@@ -46,11 +46,14 @@ async def get_sender(self):
         # cached information, they may use the property instead.
         if (self._sender is None or getattr(self._sender, 'min', None)) \
                 and await self.get_input_sender():
-            try:
-                self._sender =\
-                    await self._client.get_entity(self._input_sender)
-            except ValueError:
-                await self._refetch_sender()
+            # self.get_input_sender may refresh in which case the sender may no longer be min
+            # However it could still incur a cost so the cheap check is done twice instead.
+            if self._sender is None or getattr(self._sender, 'min', None):
+                try:
+                    self._sender =\
+                        await self._client.get_entity(self._input_sender)
+                except ValueError:
+                    await self._refetch_sender()
         return self._sender
 
     @property