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

Exceptions while parsing EDUs mask the problem area of the source #3297

Open
turt2live opened this issue May 28, 2018 · 0 comments
Open

Exceptions while parsing EDUs mask the problem area of the source #3297

turt2live opened this issue May 28, 2018 · 0 comments
Labels
O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Minor significance, cosmetic issues, low or no impact to users. T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.

Comments

@turt2live
Copy link
Member

Description

Received this error on dev.t2bot.io:

2018-05-27 20:20:21,991 - synapse.federation.transport.server - 148 - INFO - PUT-4244- Request from zemos.net
2018-05-27 20:20:21,991 - synapse.federation.transport.server - 266 - INFO - PUT-4244- Received txn DxKPPsdBfaB6LZyKgAJ6TvpMpj8Lzfp6Qs8RkmHSpp6ot from zemos.net. (PDUs: 0, EDUs: 1, failures: 0)
2018-05-27 20:20:21,996 - synapse.federation.federation_server - 643 - ERROR - PUT-4244- Failed to handle edu 'm.typing'
Traceback (most recent call last):
  File "/home/travis/.synapse/local/lib/python2.7/site-packages/synapse/federation/federation_server.py", line 639, in on_edu
    yield handler(origin, content)
KeyError: 'room_id'

Turns out the error originated from

room_id = content["room_id"]

Would be nice to see that in the stack trace at the least, to avoid having to reverse engineer the code.

Version information

  • Homeserver: dev.t2bot.io
  • Version: 0.30.0
  • Install method: pip
  • Platform: Ubuntu 16.04 VM on ESXi 6.5
@neilisfragile neilisfragile added z-p2 (Deprecated Label) z-minor (Deprecated Label) labels Jun 4, 2018
@H-Shay H-Shay added T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Minor significance, cosmetic issues, low or no impact to users. and removed z-p2 (Deprecated Label) z-minor (Deprecated Label) labels Mar 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
O-Occasional Affects or can be seen by some users regularly or most users rarely S-Tolerable Minor significance, cosmetic issues, low or no impact to users. T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.
Projects
None yet
Development

No branches or pull requests

3 participants