search

getgrav / grav-learn

Grav Learn (exhaustive grav documentation)
http://learn.getgrav.org
MIT License
228 stars 786 forks source link

Add http cache header to nginx #899

Closed pierrejoye closed 3 years ago

mahagr commented 3 years ago

I would set a shorter cache time for more dynamic assets, but it's a good addition. :)

pierrejoye commented 3 years ago

Agreed Matias, I use this one as I add hashes to each, so each version will have new cache and older versions remain. As long as the html and other links use the new ones in their content, it is all good. That should work as well when assets pipelines has enable_asset_timestamp set. I will see if I can add a note to this section (and change the section header, H1 now, not good :).

On Fri, Feb 5, 2021 at 4:00 PM Matias Griese notifications@github.com wrote:

I would set a shorter cache time for more dynamic assets, but it's a good addition. :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/getgrav/grav-learn/pull/899#issuecomment-773894845, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACE6KEJMXHNXOR2H6QSBY3S5OXTBANCNFSM4XB4VPFA .

-- Pierre

@pierrejoye | http://www.libgd.org