Make WordPress Core

Opened 11 years ago

Closed 9 years ago

#1381 closed defect (bug) (invalid)

Optionally disable the 304 header due to incompatible aggregators

Reported by: jrconlin Owned by:
Milestone: Priority: normal
Severity: trivial Version: 1.5.1
Component: Template Keywords:
Focuses: Cc:


Several popular RSS aggregators (Sage for example) currently don't handle 304's correctly. Instead, they report a URL error and toss out the previously gathered content. I've filed a like bug with that team, but since I have plenty of bandwidth to burn, I've made the 304 report optional:

if (!DISPLAY_OLD_CONTENT) { -- jrc
if ( ($client_last_modified && $client_etag) ?
header('HTTP/1.x 304 Not Modified');
} Line: 152 --jrc

DISPLAY_OLD_CONTENT is defined in wp-config.php so I can re-enable it in the future.

Change History (2)

#1 @jrconlin
11 years ago

  • Patch set to No

#2 @mdawaffe
9 years ago

  • Resolution set to invalid
  • Status changed from new to closed

If the client is doing a conditional GET, it needs to understand the response.

Marking as invalid (not a WP problem). Please feel free to discuss.

Note: See TracTickets for help on using tickets.