It seems unnecessarily complex that the "X-Drupal-Cache: HIT" and "X-Drupal-Cache: MISS" headers are sent in different bootstrap phases. When they were first added in revision 1.276 in #147310: Implement better cache headers for reverse proxies they were both sent in DRUPAL_BOOTSTRAP_LATE_PAGE_CACHE, but since then they have "drifted". Let's bring them back together.

CommentFileSizeAuthor
x-drupal-cache-1.patch2.42 KBc960657
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dries’s picture

Status: Needs review » Fixed

This looks good to me. Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix

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