Skip to content

Commit

Permalink
Merge pull request #48 from mckinseyacademy/MCKIN-21719
Browse files Browse the repository at this point in the history
MCKIN-21719 Fixed 500 instead of 404 on proxy_content
  • Loading branch information
nasirhjafri authored Jul 16, 2020
2 parents 1857bda + 712f51a commit caaeb46
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion scormxblock/scormxblock.py
Original file line number Diff line number Diff line change
Expand Up @@ -494,7 +494,7 @@ def proxy_content(self, request, suffix=''):
if ext in mimetypes.types_map:
content_type = mimetypes.types_map[ext]
else:
return Response('Did not exist in storage: ' + path_to_file + '\nstorage.path=' + storage.path(''), status=404, content_type='text/html', charset='UTF-8')
return Response('Did not exist in storage: ' + path_to_file , status=404, content_type='text/html', charset='UTF-8')
return Response(contents, content_type=content_type)

def generate_report_data(self, user_state_iterator, limit_responses=None):
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ def package_data(pkg, roots):

setup(
name='scormxblock-xblock',
version='2.1',
version='2.2',
description='XBlock to integrate SCORM content packages',
packages=[
'scormxblock',
Expand Down

0 comments on commit caaeb46

Please sign in to comment.