Skip to content
This repository has been archived by the owner on Jan 3, 2019. It is now read-only.

cache static resources 1 day #85

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

cache static resources 1 day #85

wants to merge 1 commit into from

Conversation

J0WI
Copy link
Contributor

@J0WI J0WI commented Sep 11, 2016

No description provided.

@The-Compiler
Copy link
Member

I don't really know Apache, so someone else will have to look at this one 😉

@dbrgn
Copy link
Member

dbrgn commented Sep 27, 2016

If cache invalidation is done properly, we should choose a higher value, e.g. 3 months.

If cache invalidation does not work properly, then 1 day of caching is already too much of a hassle.

I'll try to have a look at the config another day if possible :)

@J0WI
Copy link
Contributor Author

J0WI commented Jan 19, 2018

@dbrgn did you find time to have a look at this? :)

@J0WI J0WI changed the title cache static static resources 1 day cache static resources 1 day Mar 30, 2018
@gnurbs
Copy link
Contributor

gnurbs commented Jun 20, 2018

I'd say this needs to be reviewed by webmaster.

@dbrgn
Copy link
Member

dbrgn commented Jun 20, 2018

To me it does not look like the Apache module sets those headers conditionally. The question is whether Apache handles stuff like if-changed-since or etag.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants