Hello
When a request for /favicon.ico comes into the site, it appears to never get cached.
This looks to be due to there being no Cache-Control headers returned.
We could add special cases to the caching proxy (we are using Varnish) but generally speaking I would expect this to be controlled from Drupal much like with the current tag-based approached. It would be very useful if the response could either use cache tags so that any caching proxy supporting tag-based caching can work perfectly, and the module can even flush those cache when icons change. Or alternatively to have a configuration to allow setting a max-age Cache-Control header on the returned icons.
Regards,
Jason
Needs review
1.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
I have finally decided to use the default Drupal browser cache max-age defined in System Performance.
The new cache refresh suffix ( ✨ Add suffix to icons URLs for better browser cache handling Active ) will allow refreshing earlier if needed.
Automatically closed - issue fixed for 2 weeks with no activity.