Merge branch 'master' into neel/remove-relay-aws

Author: antonpirker

Makefile

@@ -19,7 +19,7 @@ help:
 	$(VENV_PATH)/bin/pip install tox
 
 dist: .venv
-	rm -rf dist dist-serverless build
+	rm -rf dist build
 	$(VENV_PATH)/bin/pip install wheel
 	$(VENV_PATH)/bin/python setup.py sdist bdist_wheel
 .PHONY: dist

sentry_sdk/integrations/boto3.py

@@ -75,6 +75,7 @@ def _sentry_request_created(service_id, request, operation_name, **kwargs):
     span.set_data("aws.request.url", parsed_url.url)
     span.set_data(SPANDATA.HTTP_QUERY, parsed_url.query)
     span.set_data(SPANDATA.HTTP_FRAGMENT, parsed_url.fragment)
+    span.set_data(SPANDATA.HTTP_METHOD, request.method)
 
     # We do it in order for subsequent http calls/retries be
     # attached to this span.

sentry_sdk/integrations/httpx.py

@@ -48,7 +48,7 @@ def send(self, request, **kwargs):
             op=OP.HTTP_CLIENT,
             description="%s %s" % (request.method, parsed_url.url),
         ) as span:
-            span.set_data("method", request.method)
+            span.set_data(SPANDATA.HTTP_METHOD, request.method)
             span.set_data("url", parsed_url.url)
             span.set_data(SPANDATA.HTTP_QUERY, parsed_url.query)
             span.set_data(SPANDATA.HTTP_FRAGMENT, parsed_url.fragment)
@@ -89,7 +89,7 @@ async def send(self, request, **kwargs):
             op=OP.HTTP_CLIENT,
             description="%s %s" % (request.method, parsed_url.url),
         ) as span:
-            span.set_data("method", request.method)
+            span.set_data(SPANDATA.HTTP_METHOD, request.method)
             span.set_data("url", parsed_url.url)
             span.set_data(SPANDATA.HTTP_QUERY, parsed_url.query)
             span.set_data(SPANDATA.HTTP_FRAGMENT, parsed_url.fragment)

sentry_sdk/integrations/stdlib.py

@@ -91,7 +91,7 @@ def putrequest(self, method, url, *args, **kwargs):
             description="%s %s" % (method, parsed_url.url),
         )
 
-        span.set_data("method", method)
+        span.set_data(SPANDATA.HTTP_METHOD, method)
         span.set_data("url", parsed_url.url)
         span.set_data(SPANDATA.HTTP_QUERY, parsed_url.query)
         span.set_data(SPANDATA.HTTP_FRAGMENT, parsed_url.fragment)

setup.py

@@ -41,6 +41,7 @@ def get_file_text(file_name):
         'urllib3>=1.25.7; python_version<="3.4"',
         'urllib3>=1.26.9; python_version=="3.5"',
         'urllib3>=1.26.11; python_version >="3.6"',
+        'urllib3<2.0.0',
         "certifi",
     ],
     extras_require={

tests/integrations/httpx/test_httpx.py

@@ -43,7 +43,7 @@ def before_breadcrumb(crumb, hint):
         assert crumb["category"] == "httplib"
         assert crumb["data"] == {
             "url": url,
-            "method": "GET",
+            SPANDATA.HTTP_METHOD: "GET",
             SPANDATA.HTTP_FRAGMENT: "",
             SPANDATA.HTTP_QUERY: "",
             "status_code": 200,

tests/integrations/requests/test_requests.py

@@ -25,7 +25,7 @@ def test_crumb_capture(sentry_init, capture_events):
     assert crumb["category"] == "httplib"
     assert crumb["data"] == {
         "url": url,
-        "method": "GET",
+        SPANDATA.HTTP_METHOD: "GET",
         SPANDATA.HTTP_FRAGMENT: "",
         SPANDATA.HTTP_QUERY: "",
         "status_code": response.status_code,

tests/integrations/stdlib/test_httplib.py

@@ -48,7 +48,7 @@ def test_crumb_capture(sentry_init, capture_events):
     assert crumb["category"] == "httplib"
     assert crumb["data"] == {
         "url": url,
-        "method": "GET",
+        SPANDATA.HTTP_METHOD: "GET",
         "status_code": 200,
         "reason": "OK",
         SPANDATA.HTTP_FRAGMENT: "",
@@ -75,7 +75,7 @@ def before_breadcrumb(crumb, hint):
     assert crumb["category"] == "httplib"
     assert crumb["data"] == {
         "url": url,
-        "method": "GET",
+        SPANDATA.HTTP_METHOD: "GET",
         "status_code": 200,
         "reason": "OK",
         "extra": "foo",
@@ -133,7 +133,7 @@ def test_httplib_misuse(sentry_init, capture_events, request):
     assert crumb["category"] == "httplib"
     assert crumb["data"] == {
         "url": "http://localhost:{}/200".format(PORT),
-        "method": "GET",
+        SPANDATA.HTTP_METHOD: "GET",
         "status_code": 200,
         "reason": "OK",
         SPANDATA.HTTP_FRAGMENT: "",