matrix-org / sygnal

Sygnal: reference Push Gateway for Matrix
Apache License 2.0
164 stars 147 forks source link

Extend logging for outgoing pushes #334

Closed richvdh closed 1 year ago

richvdh commented 1 year ago

A bit more logging is useful for being able to diagnose issues. Log the room id and event id for each push, where we have them.

DMRobertson commented 1 year ago

(No idea why flake8 has fallen over. Presumably we don't pin something that we ought to.)

richvdh commented 1 year ago

(No idea why flake8 has fallen over. Presumably we don't pin something that we ought to.)

well, I'm not in a position to investigate myself, and it's blocking the tests. Are you happy for me to mash the merge button without running the tests, or?

DMRobertson commented 1 year ago

(No idea why flake8 has fallen over. Presumably we don't pin something that we ought to.)

well, I'm not in a position to investigate myself, and it's blocking the tests. Are you happy for me to mash the merge button without running the tests, or?

Mash it aggressively; I will to unsadden Sygnal today.