Problem obtaining the Feed of an archive of tag “RSS”

[xavivars]

I'm using RSS as a tag for some of my blog posts, and I was using them without any problem.

myblog.com/tag/rss/

But I've just realised that Google Webmaster Tools gives me some related errors.

The issue is this: when trying to retrieve the feed of this tag, from

myblog.com/tag/rss/feed/

it gets redirected to

myblog.com/tag/feed/

And this gives an error.

[SergeyBiryukov]

Same for "feed", "rss2", "rdf", and "atom" tags.

Caused by stripping feed endings in redirect_canonical():
​http://core.trac.wordpress.org/browser/tags/3.4.1/wp-includes/canonical.php#L221

Related: [9697] (for #8076), [10505] (for #8642).

[SergeyBiryukov]

A workaround is to disable canonical redirection for tag feeds:

function disable_canonical_redirection_for_tag_feeds() {
	if ( is_feed() && is_tag() )
		remove_action( 'template_redirect', 'redirect_canonical' );
}
add_action( 'wp', 'disable_canonical_redirection_for_tag_feeds' );

[xavivars]

Thanks for the workaroud.

Do you know how is the process to fix this bug? It's the first time I submit one, and I'm not sure how does it work.

I mean, a developer has to assign the ticket to himself?

[SergeyBiryukov]

Not necessarily, anyone can write a patch.

However, the issue seems tricky and will require unit test coverage:
​http://make.wordpress.org/core/handbook/automated-testing/

[SergeyBiryukov]

Related: #16133

[wonderboymusic]

This is not just tags / tax, it's any query var that matches a feed slug, my patch addresses them. Attached.

[wonderboymusic]

It seems that/tag/rss/feed/ works now in trunk, but /tag/rss/feed breaks on redirection. My patch is refreshed, makes /tag/rss/feed redirect to /tag/rss/feed/ properly

[atimmer]

​test.21700.1.diff adds 4 tests for using tag's with the same name as feed names. hmm, wondering if it needs a test with custom feed names too.

The tests indicate that the comment of wonderboymusic is correct. His patch doesn't fix the tests though.

[duck_]

Unfortunately the problem is more extensive so the current patch isn't enough. This is because stripping occurs for more than just feeds. For example, /tag/rss/page/2/ becomes /tag/page/2/ with and without the patch.

[atimmer]

​tests.21700.2.diff adds the test duck_ suggested.

[nacin]

Replying to duck_:

Unfortunately the problem is more extensive so the current patch isn't enough. This is because stripping occurs for more than just feeds. For example, /tag/rss/page/2/ becomes /tag/page/2/ with and without the patch.

[chriscct7]

Per comment:13