summary refs log tree commit diff
path: root/synapse/rest/media/v1/download_resource.py
diff options
context:
space:
mode:
authorMark Haines <mjark@negativecurvature.net>2015-04-21 17:01:04 +0100
committerMark Haines <mjark@negativecurvature.net>2015-04-21 17:01:04 +0100
commit8b183781cb3bf07980f76b84d463d50965e09047 (patch)
tree247b5fe014113d7457b6acdc83a38a2ebcc00af8 /synapse/rest/media/v1/download_resource.py
parentMerge pull request #127 from matrix-org/bugs/SYN-350 (diff)
parentSet a version_string in BaseMediaResource so that the request_handler wrapper... (diff)
downloadsynapse-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.py12
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: