feat(socialaccount): Add timeout to requests

Author: pennersr

ChangeLog.rst

@@ -19,6 +19,9 @@ Note worthy changes
 - Added a new app, user sessions, allowing users to view a list of all their
   active sessions, as well as offering a means to end these sessions.
 
+- A configurable timeout (``SOCIALACCOUNT_REQUESTS_TIMEOUT``) is now applied to
+  all upstream requests.
+
 
 Backwards incompatible changes
 ------------------------------

allauth/socialaccount/adapter.py

@@ -1,3 +1,4 @@
+import functools
 import requests
 import warnings
 
@@ -293,7 +294,11 @@ def get_app(self, request, provider, client_id=None):
         return apps[0]
 
     def get_requests_session(self):
-        return requests.Session()
+        session = requests.Session()
+        session.request = functools.partial(
+            session.request, timeout=app_settings.REQUESTS_TIMEOUT
+        )
+        return session
 
 
 def get_adapter(request=None):

allauth/socialaccount/app_settings.py

@@ -139,6 +139,10 @@ def UID_MAX_LENGTH(self):
     def SOCIALACCOUNT_STR(self):
         return self._setting("SOCIALACCOUNT_STR", None)
 
+    @property
+    def REQUESTS_TIMEOUT(self):
+        return self._setting("REQUESTS_TIMEOUT", 5)
+
 
 _app_settings = AppSettings("SOCIALACCOUNT_")
 

docs/socialaccount/configuration.rst

@@ -70,6 +70,9 @@ Available settings:
 ``SOCIALACCOUNT_PROVIDERS`` (default: ``{}``)
   Dictionary containing `provider specific settings <provider_configuration.html>`__.
 
+``SOCIALACCOUNT_REQUESTS_TIMEOUT`` (default: ``5``)
+  The timeout applied when performing upstream requests.
+
 ``SOCIALACCOUNT_QUERY_EMAIL`` (default: ``ACCOUNT_EMAIL_REQUIRED``)
   Request email address from 3rd party account provider? E.g. using
   OpenID AX, or the Facebook "email" permission.