Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#16704 closed defect (bug) (fixed)

Do not do canonical redirects for unregistered feeds

Reported by: ryan's profile ryan Owned by:
Milestone: 3.1.1 Priority: normal
Severity: normal Version: 3.1
Component: Canonical Keywords: has-patch
Focuses: Cc:


Canonical redirects will break unregistered feeds since they are not includes in the rewrite rules. Several plugins go underneath instead of using add_feed().

Discovered here:

Attachments (1)

16704.diff (497 bytes) - added by ryan 13 years ago.

Download all attachments as: .zip

Change History (6)

#1 @ryan
13 years ago

  • Component changed from General to Canonical
  • Milestone changed from Awaiting Review to 3.1.1
  • Version set to 3.1

13 years ago

#2 @ryan
13 years ago

  • Keywords has-patch added

#3 @ryan
13 years ago

Looking good so far with this test plan:

  • Visit /feed/ and /category/uncategorized/feed/, verify no redirect
  • Visit ?feed=rss2 and /category/uncategorized/?feed=rss2, verify canonical redirect to /feed/...
  • Do 1 and 2 above with a non-default core feed like atom.
  • Register a feed and do 1 and 2 above.
  • Visit an unregistered feed and verify that there is no canonical redirect and that the "not a valid feed template" message is displayed.

#4 @ryan
13 years ago

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

(In [17498]) Do not do canonical redirects for unregistered feeds. fixes #16704 for trunk

#5 @ryan
13 years ago

(In [17499]) Do not do canonical redirects for unregistered feeds. fixes #16704 for 3.1

Note: See TracTickets for help on using tickets.