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

Enable browser-side caching #2956

Closed
CrazyPython opened this issue Jun 19, 2017 · 3 comments
Closed

Enable browser-side caching #2956

CrazyPython opened this issue Jun 19, 2017 · 3 comments
Labels
Improvement Minor improvement to code Needed: design decision A core team decision is required

Comments

@CrazyPython
Copy link

I'm not sure which repository to open this on, but according to Google PageSpeed Insights, these resources are missing Cache-control headers:

https://docs.readthedocs.io/en/latest/_static/js/modernizr.min.js (expiration not specified)
https://docs.readthedocs.io/en/latest/_static/logo.svg (expiration not specified)
https://docs.readthedocs.io/en/latest/_static/readthedocs-data.js (expiration not specified)
https://docs.readthedocs.io/…t/_static/readthedocs-dynamic-include.js (expiration not specified)

Readthedocs is quite slow as of now, and this would help improve its performance.

@ghost ghost added Improvement Minor improvement to code Needed: design decision A core team decision is required labels Jul 23, 2017
@ghost
Copy link

ghost commented Jul 23, 2017

Hey, @,

I'll be adding the Enhancement and Needed: design decision label to this ticket with the triage process. As you could imagine, ReadTheDocs receives an extraordinary amount of tickets and issues through GitHub every week, and because of the smaller team, your request may take on the upside of several days to be processed and reviewed by the team.

Thank you for your patience,
ItsPugle - ReadTheDocs Triage Team

@markrwilliams
Copy link

Readthedocs should also allow caching of Intersphinx objects.inv files. Even a short max-age of 10 to 15 minutes could help reduce the load imposed by busy CI servers.

I'd be happy to contribute this change.

@humitos
Copy link
Member

humitos commented Nov 15, 2018

Closing in favor of #2955

@humitos humitos closed this as completed Nov 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Minor improvement to code Needed: design decision A core team decision is required
Projects
None yet
Development

No branches or pull requests

3 participants