Make WordPress Core

Opened 14 years ago

Closed 11 years ago

#18505 closed defect (bug) (fixed)

http 404 sent for the RSS feed when there are no posts - should be 200

Reported by: mark-k's profile mark-k Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.0 Priority: normal
Severity: normal Version: 3.2.1
Component: Feeds Keywords: has-patch commit
Focuses: Cc:

Description

for sites with no posts, the HTTP status code returned for the RSS feed is 404, but since the address was a valid address and it just happens that the content is empty, a 200 status should be sent.

(It annoys link checkers which then report that there are broken link at the site because by default for most themes there is a link to the RSS feed in the header)

Attachments (2)

18505.patch (772 bytes) - added by SergeyBiryukov 14 years ago.
18505.2.patch (413 bytes) - added by SergeyBiryukov 11 years ago.

Download all attachments as: .zip

Change History (8)

#1 @johnbillion
14 years ago

Related: #17316 and its subsequent changeset at [18192].

I wonder if including is_feed() in that conditional block will fix this issue?

#2 @SergeyBiryukov
14 years ago

  • Keywords has-patch added

#3 @SergeyBiryukov
11 years ago

#27146 was marked as a duplicate.

#4 @SergeyBiryukov
11 years ago

#28508 was marked as a duplicate.

#5 @SergeyBiryukov
11 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 4.0

#6 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 29216:

Don't 404 for empty feeds.

fixes #18505.

Note: See TracTickets for help on using tickets.