Fix #387: recursive error logging for non-http events

rollbar/contrib/starlette/requests.py

@@ -46,13 +46,7 @@ def get_current_request() -> Optional[Request]:
         )
         return None
 
-    request = _current_request.get()
-
-    if request is None:
-        log.error('Request is not available in the present context.')
-        return None
-
-    return request
+    return _current_request.get()
 
 
 def store_current_request(
@@ -63,8 +57,10 @@ def store_current_request(
 
     if receive is None:
         request = request_or_scope
-    else:
+    elif request_or_scope['type'] == 'http':
         request = Request(request_or_scope, receive)
+    else:
+        request = None
 
     _current_request.set(request)
     return request

rollbar/test/starlette_tests/test_requests.py

@@ -51,7 +51,7 @@ def test_should_accept_request_param(self):
 
         self.assertEqual(request, stored_request)
 
-    def test_should_accept_scope_and_receive_params(self):
+    def test_should_accept_scope_param_if_http_type(self):
         from starlette.requests import Request
         from rollbar.contrib.starlette.requests import store_current_request
         from rollbar.lib._async import async_receive
@@ -83,6 +83,16 @@ def test_should_accept_scope_and_receive_params(self):
 
         self.assertEqual(request, expected_request)
 
+    def test_should_not_accept_scope_param_if_not_http_type(self):
+        from rollbar.contrib.starlette.requests import store_current_request
+
+        scope = {'asgi': {'spec_version': '2.0', 'version': '3.0'}, 'type': 'lifespan'}
+        receive = {}
+
+        request = store_current_request(scope, receive)
+
+        self.assertIsNone(request)
+
     def test_hasuser(self):
         from starlette.requests import Request
         from rollbar.contrib.starlette.requests import hasuser

setup.py

@@ -1,6 +1,5 @@
 import re
 import os.path
-import sys
 from setuptools import setup, find_packages
 
 HERE = os.path.abspath(os.path.dirname(__file__))
@@ -78,8 +77,20 @@
         "Topic :: System :: Monitoring",
         ],
     install_requires=[
-        'requests>=0.12.1',
+        # The currently used version of `setuptools` has a bug,
+        # so the version requirements are not properly respected.
+        #
+        # In the current version, `requests>= 0.12.1`
+        # always installs the latest version of the package.
+        'requests>=0.12.1; python_version == "2.7"',
+        'requests>=0.12.1; python_version >= "3.6"',
+        'requests<2.26,>=0.12.1; python_version == "3.5"',
+        'requests<2.22,>=0.12.1; python_version == "3.4"',
+        'requests<2.19,>=0.12.1; python_version == "3.3"',
+        'requests<1.2,>=0.12.1; python_version == "3.2"',
+        'requests<1.2,>=0.12.1; python_version == "3.1"',
+        'requests<1.2,>=0.12.1; python_version == "3.0"',
         'six>=1.9.0'
     ],
     tests_require=tests_require,
-    )
+)