WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#24623 closed defect (bug) (fixed)

Redirection goes wrong with an additional parameter at /?feed=rss

Reported by: sweetie089 Owned by: SergeyBiryukov
Priority: normal Milestone: 3.6
Component: Canonical Version: 3.3
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

If a permalink accesses "/?feed=rss&foo=bar" in the state of being invalid, it will redirect to "/?feed=rss2 foo=bar" and "404 Not Found" will be displayed.

procedure for reproducing:

Result:

  • "404 Not Found" is displayed.

The cause was carrying out the miswritten word of the backreference by preg_replace.
patch is miswritten word correction simply.

Attachments (2)

redirect_canonical_rss_rewrite_bug_test.patch​ (405 bytes ) - added by sweetie089 13 years ago.
redirect_canonical_rss_rewrite_bug.patch​ (680 bytes ) - added by sweetie089 13 years ago.

Download all attachments as: .zip

Change History (8)

@sweetie089
13 years ago

#1 @sweetie089
13 years ago

I am sorry, the file was mistaken.

@sweetie089
13 years ago

#2 @SergeyBiryukov
13 years ago

  • Component Feeds → Canonical
  • Milestone Awaiting Review → 3.6

Confirmed. If permalinks are disabled, /?feed=rss&withcomments=1 redirects to /?feed=rss2withcomments=1.

Introduced in [18537]. The patch and the test both look good.

#3 @SergeyBiryukov
13 years ago

In 1297/tests:

Add a test for obsolete feed redirection. props sweetie089. see #24623.

#4 @SergeyBiryukov
13 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 24502:

Use correct backreference in obsolete feed redirection code. props sweetie089. fixes #24623.

#5 @sweetie089
13 years ago

Thank you.

#6 @SergeyBiryukov
13 years ago

  • Version trunk → 3.3
Note: See TracTickets for help on using tickets.