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

Faster joins: check how incoming events are authed during the partial-state phase #12644

Closed
richvdh opened this issue May 5, 2022 · 2 comments · Fixed by #12943
Closed

Faster joins: check how incoming events are authed during the partial-state phase #12644

richvdh opened this issue May 5, 2022 · 2 comments · Fixed by #12943
Assignees
Labels
A-Federated-Join joins over federation generally suck T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Comments

@richvdh
Copy link
Member

richvdh commented May 5, 2022

I/we need to revisit how incoming events are authed while we lack full state. When I was testing, it looked like we were relying on Synapse's unspecced _update_auth_events_for_auth behaviour. If we're going to rely on that, we'll have to understand it and spec it.

A subtask of this is to figure out if we need power events upfront (eg, banned users) to get reliable state.

@anoadragon453 anoadragon453 added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label May 5, 2022
@erikjohnston
Copy link
Member

Did we not decide that we should try ripping the _update_auth_events_for_auth code out?

@richvdh
Copy link
Member Author

richvdh commented May 16, 2022

Did we not decide that we should try ripping the _update_auth_events_for_auth code out?

We talked about it back in a previous tranche of event-auth work (see #9595 (comment) etc), but I think we decided we needed to convince ourselves it wasn't important, first. The work for #9595 ended up significantly simplifying it, and documenting its interface a bit, but never got as far as completely ripping it out.

The problem we have now is that the faster-joins work appears to be relying on _update_auth_events_for_auth, so we now need to re-consider whether we still want to rip it out, or do something else for faster-joins.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Federated-Join joins over federation generally suck T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants