Commit 723b49cb authored by Sofia Papagiannaki's avatar Sofia Papagiannaki
Browse files

pithos: Optionally open object's content inline in the browser.

Refs: #5019
parent 2b5459d9
......@@ -157,11 +157,4 @@ def public_read(request, v_public):
raise faults.ItemNotFound('Object does not exist')
if 'Content-Disposition' not in meta:
name = v_object.rstrip('/').split('/')[-1]
if not name:
name = v_public
meta['Content-Disposition'] = 'attachment; filename=%s' % (name,)
return object_data_response(request, sizes, hashmaps, meta, True)
......@@ -225,7 +225,8 @@ def get_object_headers(request):
return content_type, meta, get_sharing(request), get_public(request)
def put_object_headers(response, meta, restricted=False, token=None):
def put_object_headers(response, meta, restricted=False, token=None,
response['ETag'] = meta['hash'] if not UPDATE_MD5 else meta['checksum']
response['Content-Length'] = meta['bytes']
response.override_serialization = True
......@@ -255,6 +256,11 @@ def put_object_headers(response, meta, restricted=False, token=None):
for k in ('Content-Encoding', 'Content-Disposition'):
if k in meta:
response[k] = smart_str(meta[k], strings_only=True)
disposition_type = disposition_type if disposition_type in \
('inline', 'attachment') else None
if disposition_type is not None:
response['Content-Disposition'] = '%s; filename=%s' % (
disposition_type, meta['name'])
def update_manifest_meta(request, v_account, meta):
......@@ -945,7 +951,8 @@ def object_data_response(request, sizes, hashmaps, meta, public=False):
response = HttpResponse(wrapper, status=ret)
response, meta, restricted=public,
token=getattr(request, 'token', None))
token=getattr(request, 'token', None),
if ret == 206:
if len(ranges) == 1:
offset, length = ranges[0]
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment