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

Make cache time configurable per endpoint #116

Merged
merged 1 commit into from
Oct 21, 2019

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Sep 26, 2019

Before all endpoints had the same cache time. But search and categories should have a shorter cache time then the catchAll endpoint which serves the static files.

@ruflin ruflin force-pushed the different-cache-headers branch from 1fdd46e to 6a93979 Compare September 26, 2019 06:35
@ruflin ruflin force-pushed the different-cache-headers branch from 6a93979 to 128b0a8 Compare October 21, 2019 06:59
@ruflin ruflin requested review from jfsiii and skh October 21, 2019 06:59
Before all endpoints had the same cache time. But search and categories should have a shorter cache time then the catchAll endpoint which serves the static files.
@ruflin ruflin force-pushed the different-cache-headers branch from 128b0a8 to 860ac71 Compare October 21, 2019 07:20
Copy link
Contributor

@skh skh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as described 👍

@ruflin ruflin merged commit 32abb5c into elastic:master Oct 21, 2019
@ruflin ruflin deleted the different-cache-headers branch October 21, 2019 12:08
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.

2 participants