Unverified Commit d63b66d3 authored by Diederik van der Boor's avatar Diederik van der Boor
Browse files

Fix HEAD handling in combination with If-Modified-Since

parent bde30d66
...@@ -118,8 +118,9 @@ class DjangoServer(DjangoStreamingServer): ...@@ -118,8 +118,9 @@ class DjangoServer(DjangoStreamingServer):
else: else:
# Using Django's serve gives If-Modified-Since support out of the box. # Using Django's serve gives If-Modified-Since support out of the box.
response = serve(private_file.request, full_path, document_root='/', show_indexes=False) response = serve(private_file.request, full_path, document_root='/', show_indexes=False)
if private_file.request.method == 'HEAD': if private_file.request.method == 'HEAD' and response.status_code == 200:
# Avoid reading the file at all, copy FileResponse headers # Avoid reading the file at all, copy FileResponse headers
# This is not needed for HttpResponseNotModified(), hence the 200 code check
head_response = HttpResponse(status=response.status_code) head_response = HttpResponse(status=response.status_code)
for header, value in response.items(): for header, value in response.items():
head_response[header] = value head_response[header] = value
......
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