docs-matrix-spec/proposals/2451-remove-query_auth-federation-endpoint.md
2020-03-03 16:20:13 -05:00

1.6 KiB

MSC2451: Remove the query_auth federation endpoint

The query_auth federation endpoint is unused by Synapse and should be removed. The current implementation in Synapse is not robust and will return a 500 error in some situations.

Proposal

Remove the following endpoint:

Potential issues

Removing this endpoint impacts backwards compatibility, in practice removing this endpoint should have minimal impact as it was an unused error path in Synapse. The federation client code to call this endpoint was removed in Synapse v1.5.0rc1.

Note that dendrite has never implemented this federation endpoint.

History

This endpoint (and the federation client code) to call it was initially added in Synapse v0.7.0 (see #43). The federation client code was heavily modified for v1.0.0rc1 (see #5227),

The federation client code to call this endpoint was removed in v1.5.0rc1 of Synapse (see #6214. After that point this endpoint is not called).

During removal it was noted that the code to call this endpoint was already unreachable. It seems that this code was never reachable and was meant for an error situation which was never built out (see git log -S NOT_ANCESTOR, the error condition is never assigned).

Alternatives

The endpoint could be deprecated in removed in a future version of the specification.

Security considerations

None.