diff options
| author | Richard van der Hoff <github@rvanderhoff.org.uk> | 2017-10-23 14:47:49 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-10-23 14:47:49 +0100 |
| commit | ecf198aab8c92be436051fe791e0a7501e51e767 (patch) | |
| tree | 82f4048e673a3de4c26a2632a69aba90adcfecd0 /synapse/rest/media/v1 | |
| parent | Merge pull request #2561 from matrix-org/rav/id_checking (diff) | |
| parent | Fix a logcontext leak in the media repo (diff) | |
| download | synapse-ecf198aab8c92be436051fe791e0a7501e51e767.tar.xz | |
Merge pull request #2566 from matrix-org/rav/media_logcontext_leak
Fix a logcontext leak in the media repo
Diffstat (limited to 'synapse/rest/media/v1')
| -rw-r--r-- | synapse/rest/media/v1/_base.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/synapse/rest/media/v1/_base.py b/synapse/rest/media/v1/_base.py index b9600f2167..bce077becf 100644 --- a/synapse/rest/media/v1/_base.py +++ b/synapse/rest/media/v1/_base.py @@ -17,6 +17,7 @@ from synapse.http.server import respond_with_json, finish_request from synapse.api.errors import ( cs_error, Codes, SynapseError ) +from synapse.util import logcontext from twisted.internet import defer from twisted.protocols.basic import FileSender @@ -103,7 +104,9 @@ def respond_with_file(request, media_type, file_path, ) with open(file_path, "rb") as f: - yield FileSender().beginFileTransfer(f, request) + yield logcontext.make_deferred_yieldable( + FileSender().beginFileTransfer(f, request) + ) finish_request(request) else: |