Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Studio View for Scorm files is having cache issue. #1

Closed
bvenkatr opened this issue Apr 28, 2020 · 1 comment
Closed

Studio View for Scorm files is having cache issue. #1

bvenkatr opened this issue Apr 28, 2020 · 1 comment
Labels
bug Something isn't working

Comments

@bvenkatr
Copy link

I was experimenting with this lib and found that when we reupload scorm_2.zip file in same scorm component/unit in studio where scorm_1.zip was uploaded earlier, studio view is still showing/playing scorm_1.zip file content.

But when we refresh browser then it’s playing scorm_2.zip file content. It looks like there is some caching issue over there.

Here 2 sample file to test with
Women_in_Sports_Digital_Mag scorm 1.zip
Quiz scorm 1.zip

@regisb regisb added the bug Something isn't working label Apr 29, 2020
@regisb
Copy link
Contributor

regisb commented Apr 29, 2020

This issue is due to the fact that nginx caches media files with a fixed expiration time. It's difficult to modify the expiration time just for scorm files, because the scorm file location may be customized by every user (see LOCATION xblock setting).

This is problematic because the root cause is at the core of the design of the xblock. We could solve this by making sure that the package file sha1 checksum be added to the media path, but that would break existing xblocks. Sill, I think we can keep backward-compatibility, but I need to investigate a little further.

@regisb regisb closed this as completed in 48d8967 Apr 29, 2020
regisb added a commit to overhangio/tutor that referenced this issue Apr 29, 2020
@deepdad deepdad mentioned this issue Aug 1, 2020
ziafazal pushed a commit that referenced this issue May 27, 2024
Media files are served with a long TTL by nginx. To avoid this, we need
to store the files in a path that includes the package checksum. For
backward compatibility, we still support old-schoold paths reading.

Close #1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants