Page MenuHomePhabricator

generateSitemaps treats redirects as content pages
Closed, DeclinedPublic

Description

Author: alxndr

Description:
maintenance/generateSitemaps.php script includes redirect pages in the sitemaps it generates, but from a SEO standpoint they should not be included.


Version: 1.11.x
Severity: enhancement

Details

Reference
bz11319

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 21 2014, 9:58 PM
bzimport set Reference to bz11319.
bzimport added a subscriber: Unknown Object (MLST).

alxndr wrote:

quick but resource-heavy fix

This appears to work, but is incredibly time-consuming; it creates a new Article object for every title.

Attached:

alxndr wrote:

quick little benchmark for the patch, running on for 2578 URLs:

before patch: running time < 10 sec
after patch: running time = 17 minutes

...ouch

Redirects need to appear for proper indexing.