diff options
author | Mark Haines <mjark@negativecurvature.net> | 2015-04-21 17:01:04 +0100 |
---|---|---|
committer | Mark Haines <mjark@negativecurvature.net> | 2015-04-21 17:01:04 +0100 |
commit | 8b183781cb3bf07980f76b84d463d50965e09047 (patch) | |
tree | 247b5fe014113d7457b6acdc83a38a2ebcc00af8 /synapse/rest/media/v1/download_resource.py | |
parent | Merge pull request #127 from matrix-org/bugs/SYN-350 (diff) | |
parent | Set a version_string in BaseMediaResource so that the request_handler wrapper... (diff) | |
download | synapse-8b183781cb3bf07980f76b84d463d50965e09047.tar.xz |
Merge pull request #128 from matrix-org/unify_http_wrappers
Unify http wrappers
Diffstat (limited to 'synapse/rest/media/v1/download_resource.py')
-rw-r--r-- | synapse/rest/media/v1/download_resource.py | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/synapse/rest/media/v1/download_resource.py b/synapse/rest/media/v1/download_resource.py index c585bb11f7..0fe6abf647 100644 --- a/synapse/rest/media/v1/download_resource.py +++ b/synapse/rest/media/v1/download_resource.py @@ -13,7 +13,8 @@ # See the License for the specific language governing permissions and # limitations under the License. -from .base_resource import BaseMediaResource +from .base_resource import BaseMediaResource, parse_media_id +from synapse.http.server import request_handler from twisted.web.server import NOT_DONE_YET from twisted.internet import defer @@ -28,15 +29,10 @@ class DownloadResource(BaseMediaResource): self._async_render_GET(request) return NOT_DONE_YET - @BaseMediaResource.catch_errors + @request_handler @defer.inlineCallbacks def _async_render_GET(self, request): - try: - server_name, media_id = request.postpath - except: - self._respond_404(request) - return - + server_name, media_id = parse_media_id(request) if server_name == self.server_name: yield self._respond_local_file(request, media_id) else: |