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

Serve uploaded media files with nginx, not django/gunicorn #72

Merged
merged 2 commits into from
Sep 23, 2024
Merged

Conversation

yuvipanda
Copy link
Member

  • Is recommended by django for production instances (https://docs.gunicorn.org/en/latest/deploy.html)
  • Probably faster
  • Allows for future finetuning as needed
  • Does not actually change the static files - only the media files for now. Static files are in the django deployment image and more work.
  • The challenge is serving .glb files that are >10Mb, so the nginx improvement is worth it.

Ref #59

yuvipanda and others added 2 commits September 23, 2024 23:51
- Is recommended by django for production instances
  (https://docs.gunicorn.org/en/latest/deploy.html)
- Probably faster
- Allows for future finetuning as needed
- Does not actually change the *static* files - only the
  media files for now. Static files are in the django deployment
  image and more work.
- The challenge is serving .glb files that are >10Mb, so the
  nginx improvement is worth it.

Ref #59
@yuvipanda yuvipanda merged commit 138b4c3 into main Sep 23, 2024
3 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 23, 2024
@yuvipanda yuvipanda deleted the med branch November 1, 2024 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant