summary refs log tree commit diff
diff options
context:
space:
mode:
authorPatrick Cloke <clokep@users.noreply.github.com>2022-07-25 08:34:06 -0400
committerGitHub <noreply@github.com>2022-07-25 08:34:06 -0400
commit908aeac44ab3acc1c42fd6c33ef716ddf51142a7 (patch)
treea2ff79796191bc95a966a943ef6ca2c3ae4d539d
parentRefactor presence so we can prune user in room caches (#13313) (diff)
downloadsynapse-908aeac44ab3acc1c42fd6c33ef716ddf51142a7.tar.xz
Additional fixes for opentracing type hints. (#13362)
-rw-r--r--changelog.d/13362.misc1
-rw-r--r--synapse/logging/opentracing.py4
2 files changed, 3 insertions, 2 deletions
diff --git a/changelog.d/13362.misc b/changelog.d/13362.misc
new file mode 100644
index 0000000000..c80578ce95
--- /dev/null
+++ b/changelog.d/13362.misc
@@ -0,0 +1 @@
+Add missing type hints to open tracing module.
diff --git a/synapse/logging/opentracing.py b/synapse/logging/opentracing.py
index ad5cbf46a4..c1aa205eed 100644
--- a/synapse/logging/opentracing.py
+++ b/synapse/logging/opentracing.py
@@ -910,8 +910,8 @@ def tag_args(func: Callable[P, R]) -> Callable[P, R]:
     def _tag_args_inner(*args: P.args, **kwargs: P.kwargs) -> R:
         argspec = inspect.getfullargspec(func)
         for i, arg in enumerate(argspec.args[1:]):
-            set_tag("ARG_" + arg, args[i])  # type: ignore[index]
-        set_tag("args", args[len(argspec.args) :])  # type: ignore[index]
+            set_tag("ARG_" + arg, str(args[i]))  # type: ignore[index]
+        set_tag("args", str(args[len(argspec.args) :]))  # type: ignore[index]
         set_tag("kwargs", str(kwargs))
         return func(*args, **kwargs)