regression-fix: retain httpx.URL type on request.url

CHANGELOG.md

@@ -19,6 +19,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
  ([#2297](https://github.com/open-telemetry/opentelemetry-python-contrib/issues/2297))
 - Ensure all http.server.duration metrics have the same description
  ([#2151](https://github.com/open-telemetry/opentelemetry-python-contrib/issues/2298))
+- Fix regression in httpx `request.url` not being of type `httpx.URL` after `0.44b0`
+ ([#2359](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/2359))
 - Avoid losing repeated HTTP headers
  ([#2266](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/2266))
 - `opentelemetry-instrumentation-elasticsearch` Don't send bulk request body as db statement

instrumentation/opentelemetry-instrumentation-httpx/src/opentelemetry/instrumentation/httpx/__init__.py

@@ -270,7 +270,7 @@ def _extract_parameters(args, kwargs):
  # In httpx >= 0.20.0, handle_request receives a Request object
  request: httpx.Request = args[0]
  method = request.method.encode()
- url = remove_url_credentials(str(request.url))
+ url = httpx.URL(remove_url_credentials(str(request.url)))
  headers = request.headers
  stream = request.stream
  extensions = request.extensions

instrumentation/opentelemetry-instrumentation-httpx/tests/test_httpx_integration.py

@@ -51,12 +51,18 @@
 HTTP_RESPONSE_BODY = "http.response.body"
 
 
+def _is_url_tuple(request: "RequestInfo"):
+ """Determine if request url format is for httpx versions < 0.20.0."""
+ return isinstance(request[1], tuple) and len(request[1]) == 4
+
+
 def _async_call(coro: typing.Coroutine) -> asyncio.Task:
  loop = asyncio.get_event_loop()
  return loop.run_until_complete(coro)
 
 
 def _response_hook(span, request: "RequestInfo", response: "ResponseInfo"):
+ assert _is_url_tuple(request) or isinstance(request.url, httpx.URL)
  span.set_attribute(
  HTTP_RESPONSE_BODY,
  b"".join(response[2]),
@@ -66,18 +72,21 @@ def _response_hook(span, request: "RequestInfo", response: "ResponseInfo"):
 async def _async_response_hook(
  span: "Span", request: "RequestInfo", response: "ResponseInfo"
 ):
+ assert _is_url_tuple(request) or isinstance(request.url, httpx.URL)
  span.set_attribute(
  HTTP_RESPONSE_BODY,
  b"".join([part async for part in response[2]]),
  )
 
 
 def _request_hook(span: "Span", request: "RequestInfo"):
+ assert _is_url_tuple(request) or isinstance(request.url, httpx.URL)
  url = httpx.URL(request[1])
  span.update_name("GET" + str(url))
 
 
 async def _async_request_hook(span: "Span", request: "RequestInfo"):
+ assert _is_url_tuple(request) or isinstance(request.url, httpx.URL)
  url = httpx.URL(request[1])
  span.update_name("GET" + str(url))