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

Internal server error when looking up a invalid room alias  #13104

@DMRobertson

Description

@DMRobertson

https://sentry.tools.element.io/organizations/element/issues/923/?project=2&query=is%3Aunresolved

ValueError: At least one of the arguments destination and destination_is must be a nonempty bytestring.
  File "synapse/http/server.py", line 366, in _async_render_wrapper
    callback_return = await self._async_render(request)
  File "synapse/http/server.py", line 572, in _async_render
    callback_return = await raw_callback_return
  File "synapse/rest/client/directory.py", line 57, in on_GET
    res = await self.directory_handler.get_association(room_alias_obj)
  File "synapse/handlers/directory.py", line 262, in get_association
    fed_result: Optional[JsonDict] = await self.federation.make_query(
  File "synapse/federation/federation_client.py", line 183, in make_query
    return await self.transport_layer.make_query(
  File "synapse/federation/transport/client.py", line 260, in make_query
    return await self.client.get_json(
  File "synapse/http/matrixfederationclient.py", line 1069, in get_json
    response = await self._send_request_with_optional_trailing_slash(
  File "synapse/http/matrixfederationclient.py", line 405, in _send_request_with_optional_trailing_slash
    response = await self._send_request(request, **send_request_args)
  File "synapse/http/matrixfederationclient.py", line 555, in _send_request
    auth_headers = self.build_auth_headers(
  File "synapse/http/matrixfederationclient.py", line 735, in build_auth_headers
    raise ValueError(

This came from a GET request for url: /_matrix/client/r0/directory/room/%23Computer-security%3A, .

That's an invalid room alias because it doesn't contain a domain name. Very similar to #12690, then.

Metadata

Metadata

Assignees

No one assigned

    Labels

    P4(OBSOLETE: use S- labels.) Okay backlog: will not schedule, will accept patchesS-TolerableMinor significance, cosmetic issues, low or no impact to users.T-DefectBugs, crashes, hangs, security vulnerabilities, or other reported issues.good first issueGood for newcomers

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions