Set a generic user-agent for bot detection (#30)

Fixes #29

Author: ferhatelmas

CHANGELOG

@@ -1,3 +1,6 @@
+## June 23, 2020 - 1.3.1
+- Set a generic user agent for file/image get to prevent bot detection
+
 ## Apr 28, 2020 - 1.3.0
 - Drop six dependency
   - `verify_webhook` is affected and expects bytes for body parameter

stream_chat/client.py

@@ -3,6 +3,7 @@
 import hashlib
 import json
 import urllib
+from urllib.request import Request, urlopen
 
 import jwt
 import requests
@@ -302,15 +303,12 @@ def send_file(self, uri, url, name, user, content_type=None):
         parts = urlparse(url)
         if parts[0] == "":
             url = "file://" + url
-        if content_type:
-            file_tuple = (name, urllib.request.urlopen(url), content_type)
-        else:
-            file_tuple = (name, urllib.request.urlopen(url), content_type)
+        content = urlopen(Request(url, headers={"User-Agent": "Mozilla/5.0"})).read()
         response = requests.post(
             "{}/{}".format(self.base_url, uri),
             params=self.get_default_params(),
             data={"user": json.dumps(user)},
-            files={"file": file_tuple},
+            files={"file": (name, content, content_type)},
             headers=headers,
         )
         return self._parse_response(response)

stream_chat/tests/test_channel.py

@@ -147,6 +147,13 @@ def test_send_and_delete_image(self, channel, random_user):
         assert "lena.png" in resp["file"]
         # resp = channel.delete_image(resp['file'])
 
+    def test_send_image_with_bot_blocked(self, channel, random_user):
+        # following url blocks bots and we set a generic header to skip it
+        # but it can start failing again, see initial discussion here: https://github.com/GetStream/stream-chat-python/pull/30#discussion_r444209891
+        url = "https://api.twilio.com/2010-04-01/Accounts/AC3e136e1a00279f4dadcb10a9f1a1e8a3/Messages/MM547edf6f4846a30231c3033fa20f8419/Media/ME498020f8fe0b0ba2ff83ac99e4782e02"
+        resp = channel.send_image(url, "js.png", random_user, content_type="image/png")
+        assert "js.png" in resp["file"]
+
     def test_channel_hide_show(self, client, channel, random_users):
         # setup
         channel.add_members([u["id"] for u in random_users])