Make LoggingContext's name optional (#9857)

Fixes https://github.com/matrix-org/synapse-s3-storage-provider/issues/55
This commit is contained in:
Richard van der Hoff 2021-04-21 16:39:34 +01:00 committed by GitHub
parent bdb4c20dc1
commit d9bd62f9d1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 3 deletions

1
changelog.d/9857.bugfix Normal file
View file

@ -0,0 +1 @@
Fix a regression in Synapse v1.32.1 which caused `LoggingContext` errors in plugins.

View file

@ -258,7 +258,8 @@ class LoggingContext:
child to the parent
Args:
name (str): Name for the context for debugging.
name: Name for the context for logging. If this is omitted, it is
inherited from the parent context.
parent_context (LoggingContext|None): The parent of the new context
"""
@ -277,12 +278,11 @@ class LoggingContext:
def __init__(
self,
name: str,
name: Optional[str] = None,
parent_context: "Optional[LoggingContext]" = None,
request: Optional[ContextRequest] = None,
) -> None:
self.previous_context = current_context()
self.name = name
# track the resources used by this context so far
self._resource_usage = ContextResourceUsage()
@ -314,6 +314,15 @@ class LoggingContext:
# the request param overrides the request from the parent context
self.request = request
# if we don't have a `name`, but do have a parent context, use its name.
if self.parent_context and name is None:
name = str(self.parent_context)
if name is None:
raise ValueError(
"LoggingContext must be given either a name or a parent context"
)
self.name = name
def __str__(self) -> str:
return self.name