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.

Sign up for GitHub

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

Jump to bottom

Set a caching header by default #117

Open
Benjozork opened this issue Mar 21, 2020 · 1 comment
Open

Set a caching header by default #117

Benjozork opened this issue Mar 21, 2020 · 1 comment
Assignees
@Kilo59
Labels
enhancement New feature or request performance Issue related to performance and optimizations
Milestone
Performance Optim...

Comments

@Benjozork
Copy link
Contributor

Just a suggestion : Since the data is only refreshed every hour, it would be perfectly safe to set

Cache-Control: max-age=<time until next refresh, calculated on the server>

in every response. It would reduce the strain on your app.
@ExpDev07 ExpDev07 added the enhancement New feature or request label Mar 21, 2020
@ExpDev07
Copy link
Owner

Thanks! Will definitively check this out.

@Kilo59 Kilo59 added this to the Performance Optimizations milestone Mar 26, 2020
@Kilo59 Kilo59 added the performance Issue related to performance and optimizations label Mar 29, 2020
@Kilo59 Kilo59 mentioned this issue Apr 26, 2020
Closed
@Kilo59 Kilo59 self-assigned this Apr 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Kilo59 Kilo59

Labels
enhancement New feature or request performance Issue related to performance and optimizations
Projects
None yet
Milestone
Performance Optimizations
Development

No branches or pull requests
3 participants
@Benjozork @ExpDev07 @Kilo59