Here's a patch that let's drupal serve the favicon from the page-cache. It also adds some more-meaningful headers for cache-control.

CommentFileSizeAuthor
#1 favicon-support-caching.patch1.75 KBtorotil
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

torotil’s picture

torotil CreditAttribution: torotil commented
FileSize
favicon-support-caching.patch1.75 KB

Here's the patch.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Issue summary: View changes
Status: Needs review » Fixed

This should be resolved with the most recent release, 7.x-2.0. It is safe for everyone to upgrade to since there's no data that actually needs upgrading.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.