aiohttp instrumentation: correct url filter input type (open-telemetry#864)

* fix: correct _UrlFilterT

* docs: CHANGELOG.md update

* style: apply isort

* fix: add yarl to docs requirements

* Revert "fix: add yarl to docs requirements"

This reverts commit e482ba5.

* fix: add yarl to nitpick exceptions

Co-authored-by: Diego Hurtado <[email protected]>

Author: 2 people

CHANGELOG.md

@@ -40,21 +40,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - `opentelemetry-instrumentation-django` Django: Conditionally create SERVER spans
   ([#832](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/832))
-  
 - `opentelemetry-instrumentation-flask` Flask: Conditionally create SERVER spans
   ([#828](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/828))
 - `opentelemetry-instrumentation-celery` Celery: Support partial task time limit
   ([#846](https://github.com/open-telemetry/opentelemetry-python-contrib/issues/846))
-
 - `opentelemetry-instrumentation-asgi` ASGI: Conditionally create SERVER spans
   ([#843](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/843))
-
 - `opentelemetry-instrumentation-django` Django: fix issue preventing detection of MIDDLEWARE_CLASSES
-
 - `opentelemetry-instrumentation-sqlite3` Instrumentation now works with `dbapi2.connect`
-
 - `opentelemetry-instrumentation-kafka` Kafka: safe kafka partition extraction
   ([#872](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/872))
+- `opentelemetry-instrumentation-aiohttp-client` aiohttp: Correct url filter input type
+  ([#843](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/864))
+
 
 ## [1.8.0-0.27b0](https://github.com/open-telemetry/opentelemetry-python/releases/tag/v1.8.0-0.27b0) - 2021-12-17
 

docs/nitpick-exceptions.ini

@@ -25,6 +25,7 @@ class_references=
     httpx.AsyncBaseTransport
     httpx.SyncByteStream
     httpx.AsyncByteStream
+    yarl.URL
 
 anys=
     ; API

instrumentation/opentelemetry-instrumentation-aiohttp-client/src/opentelemetry/instrumentation/aiohttp_client/__init__.py

@@ -91,6 +91,7 @@ def response_hook(span: Span, params: typing.Union[
 
 import aiohttp
 import wrapt
+import yarl
 
 from opentelemetry import context as context_api
 from opentelemetry import trace
@@ -108,7 +109,7 @@ def response_hook(span: Span, params: typing.Union[
 from opentelemetry.trace.status import Status, StatusCode
 from opentelemetry.util.http import remove_url_credentials
 
-_UrlFilterT = typing.Optional[typing.Callable[[str], str]]
+_UrlFilterT = typing.Optional[typing.Callable[[yarl.URL], str]]
 _RequestHookT = typing.Optional[
     typing.Callable[[Span, aiohttp.TraceRequestStartParams], None]
 ]