Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

/turnServer doesn't specify error when homeserver has no TURN server #1795

Open
Kladki opened this issue Apr 17, 2024 · 0 comments · May be fixed by matrix-org/matrix-spec-proposals#4166
Open

/turnServer doesn't specify error when homeserver has no TURN server #1795

Kladki opened this issue Apr 17, 2024 · 0 comments · May be fixed by matrix-org/matrix-spec-proposals#4166
Labels
clarification An area where the expected behaviour is understood, but the spec could do with being more explicit

Comments

@Kladki
Copy link
Contributor

Kladki commented Apr 17, 2024 •
edited
Loading

Link to problem area:

https://spec.matrix.org/v1.10/client-server-api/#get_matrixclientv3voipturnserver

Issue

In the spec, it does not tell homeservers what to respond with for requests to /voip/turnServer when there is no TURN server configured. Currently:

  • Synapse and Dendrite return 200 OK with an empty body.
  • Conduit and Construct return empty strings for username and password, and an empty array for uris.
  • Telodendria does not support this endpoint yet.

What would be the correct behavior? The empty body, an (M_NOT_FOUND) error code, or something else?
@Kladki Kladki added the clarification An area where the expected behaviour is understood, but the spec could do with being more explicit label Apr 17, 2024
@Kladki Kladki linked a pull request Jul 6, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clarification An area where the expected behaviour is understood, but the spec could do with being more explicit
Projects
None yet
Milestone
No milestone
1 participant
@Kladki