Making WordPress.org

Opened 4 years ago

Closed 4 years ago

#5499 closed defect (bug) (reported-upstream)

XML sitemaps omit trailing slashes

Reported by: jonoaldersonwp's profile jonoaldersonwp Owned by:
Milestone: Priority: lowest
Component: General Keywords: seo close
Cc:

Description

Various XML sitemaps on wordpress.org contain references to top-level pages, which incorrectly omit a trailing slash.

E.g., the XML sitemap at https://wordpress.org/plugins/sitemap-1.xml contains a reference to https://wordpress.org/plugins. This should be https://wordpress.org/plugins/.

The same issue exists for themes (https://wordpress.org/themes/sitemap-1.xml) and news (https://wordpress.org/news/sitemap-1.xml)

Change History (6)

This ticket was mentioned in Slack in #meta by tellyworth. View the logs.


4 years ago

#2 follow-up: @tellyworth
4 years ago

Is this related to https://core.trac.wordpress.org/ticket/50570? Seems curious that didn't fix it.

#3 in reply to: ↑ 2 ; follow-up: @dd32
4 years ago

Replying to tellyworth:

Is this related to https://core.trac.wordpress.org/ticket/50570? Seems curious that didn't fix it.

Looks like we're relying upon Jetpack sitemaps, which overrides the core behaviour, and has this bug too.

We should probably switch to the core one, and this should also be filed upstream with Jetpack as we're not in a position to fix this specifically on W.org.

#4 in reply to: ↑ 3 @dd32
4 years ago

  • Keywords close added

Replying to dd32:

should also be filed upstream with Jetpack as we're not in a position to fix this specifically on W.org.

https://github.com/Automattic/jetpack/issues/17722
https://github.com/Automattic/jetpack/pull/17723

#5 @jonoaldersonwp
4 years ago

The core sitemaps aren't suitable for wp.org (without the same kinds of - and far more - modification than we're already applying to the Jetpack ones). Awesome if we can get this fixed in Jetpack, though!

#6 @dd32
4 years ago

  • Resolution set to reported-upstream
  • Status changed from new to closed

Upstream PR was merged, this will be fixed on WordPress.org with the next Jetpack update.

Note: See TracTickets for help on using tickets.