Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit b851b3f

Browse files
committed
Remove redundant parameters on _check_event_auth
as of #11012, these parameters are unused.
1 parent b09d90c commit b851b3f

File tree

3 files changed

+1
-12
lines changed

3 files changed

+1
-12
lines changed

changelog.d/11292.misc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Remove unused parameters on `FederationEventHandler._check_event_auth`.

synapse/handlers/federation_event.py

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -981,8 +981,6 @@ async def _process_received_pdu(
981981
origin,
982982
event,
983983
context,
984-
state=state,
985-
backfilled=backfilled,
986984
)
987985
except AuthError as e:
988986
# FIXME richvdh 2021/10/07 I don't think this is reachable. Let's log it
@@ -1332,8 +1330,6 @@ async def _check_event_auth(
13321330
origin: str,
13331331
event: EventBase,
13341332
context: EventContext,
1335-
state: Optional[Iterable[EventBase]] = None,
1336-
backfilled: bool = False,
13371333
) -> EventContext:
13381334
"""
13391335
Checks whether an event should be rejected (for failing auth checks).
@@ -1344,12 +1340,6 @@ async def _check_event_auth(
13441340
context:
13451341
The event context.
13461342
1347-
state:
1348-
The state events used to check the event for soft-fail. If this is
1349-
not provided the current state events will be used.
1350-
1351-
backfilled: True if the event was backfilled.
1352-
13531343
Returns:
13541344
The updated context object.
13551345

tests/test_federation.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,6 @@ async def _check_event_auth(
8181
origin,
8282
event,
8383
context,
84-
state=None,
85-
backfilled=False,
8684
):
8785
return context
8886

0 commit comments

Comments
 (0)