If content hashing is enabled on webpack, cache time can even be 10 years as the file names will update and effectively invalidate the old caches. If content hashing or some other cache busting mechanism is not enabled, it might be a problem.
In my experience, it is good to have one year, browsers usually re-download resources marked for 30 days quite often
Attached is the patch to increase cache control max age (in http header) for static files to improve performance over longer run.
By default it is 43200 seconds (12 hrs), which means that the static files like vendor.js, etc. will be fetched again from server after 12hrs. This is not required as these files won't change.
Kindly review.
I have done the review, it looks good to me.
I have only concern about - with this change the caching will be done for at least 1 year. I am still kind of confused, what should be the caching time?
