Skip to content

Remove recdirects from credits APIs#643

Closed
peterwilsoncc wants to merge 9 commits into
WordPress:trunkfrom
peterwilsoncc:remove-credit-redirects
Closed

Remove recdirects from credits APIs#643
peterwilsoncc wants to merge 9 commits into
WordPress:trunkfrom
peterwilsoncc:remove-credit-redirects

Conversation

@peterwilsoncc
Copy link
Copy Markdown

@peterwilsoncc peterwilsoncc commented May 20, 2026

Updates all the links in the credits APIs that were leading to redirects

Warning

I'm unable to get a local environment set up so someone with a sandbox will need to ensure that I haven't introduced a fatal error.

More complicated:

Note: Most changes are from http urls to https. I've trusted the bot to get these right and haven't manually tested them. Other changes have all been validated and modified if required.

AI Assistance

  • Prompt:

    Review the files in the api.wordpress.org/public_html/core/credits for the external library credits.

    Each of these include a link to the library's home page.

    Make a HEAD request for these links and docuement which ones trigger a redirect. Follow the redirects until you receive a successfull HTTP response.

    You may use curl to make the HEAD requests.

    At this stage only report, do not make any changes to files.

    Eventually followed by a request to make the changes.

  • Agent: Claude Haiku 4.5 • 0.3x

See https://meta.trac.wordpress.org/ticket/8252

@peterwilsoncc peterwilsoncc marked this pull request as ready for review May 20, 2026 23:52
Copilot AI review requested due to automatic review settings May 20, 2026 23:52
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 20, 2026

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props peterwilsoncc, dd32.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the WordPress core credits API data to use canonical (non-redirecting) URLs for user profiles and external library homepages across many historical WordPress versions, reducing unnecessary HTTP redirects for API consumers.

Changes:

  • Add a trailing slash to the profiles.wordpress.org URL template returned by the credits API.
  • Update multiple external library homepage links (e.g., React, TinyMCE, getID3, hoverIntent, jQuery Hotkeys, whatwg-fetch, Babel Polyfill) to newer/canonical destinations and/or HTTPS.
  • Apply these link updates consistently across wp-32.php through wp-70.php credits datasets.

Reviewed changes

Copilot reviewed 40 out of 40 changed files in this pull request and generated no comments.

Show a summary per file
File Description
api.wordpress.org/public_html/core/credits/wp-credits.php Add trailing slash to Profiles URL template to avoid redirects.
api.wordpress.org/public_html/core/credits/wp-70.php Update select external library URLs (jQuery Hotkeys, whatwg-fetch).
api.wordpress.org/public_html/core/credits/wp-69.php Update external library URLs (Babel Polyfill, jQuery Hotkeys, React, TinyMCE, whatwg-fetch).
api.wordpress.org/public_html/core/credits/wp-68.php Update external library URLs (Babel Polyfill, jQuery Hotkeys, React, TinyMCE, whatwg-fetch).
api.wordpress.org/public_html/core/credits/wp-67.php Update external library URLs (Babel Polyfill, jQuery Hotkeys, React, TinyMCE, whatwg-fetch).
api.wordpress.org/public_html/core/credits/wp-66.php Update external library URLs (Babel Polyfill, jQuery Hotkeys, React, TinyMCE, whatwg-fetch).
api.wordpress.org/public_html/core/credits/wp-65.php Update external library URLs (Babel Polyfill, jQuery Hotkeys, React, TinyMCE, whatwg-fetch).
api.wordpress.org/public_html/core/credits/wp-64.php Update external library URLs (Babel Polyfill, jQuery Hotkeys, React, TinyMCE, whatwg-fetch).
api.wordpress.org/public_html/core/credits/wp-63.php Update external library URLs (Babel Polyfill, jQuery Hotkeys, React, TinyMCE, whatwg-fetch).
api.wordpress.org/public_html/core/credits/wp-62.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-61.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-60.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-59.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-58.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-57.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-56.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-55.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-54.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-53.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-52.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-51.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-50.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, Requests, Thickbox, TinyMCE, React, etc.).
api.wordpress.org/public_html/core/credits/wp-49.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, PclZip, Requests, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-48.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, PclZip, Requests, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-47.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, PclZip, Requests, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-46.php Update multiple external library URLs (incl. getID3, hoverIntent, Masonry, MediaElement, PclZip, Requests, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-45.php Update multiple external library URLs (incl. getID3, jQuery/jQuery UI, hoverIntent, Masonry, MediaElement, PclZip, phpass, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-44.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, jQuery Hotkeys, PHPMailer, Masonry, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-43.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, jQuery Hotkeys, PHPMailer, Masonry, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-42.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, jQuery Hotkeys, PHPMailer, Masonry, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-41.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, jQuery Hotkeys, PHPMailer, Masonry, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-40.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, PHPMailer, Masonry, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-39.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, PHPMailer, Masonry, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-38.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, PHPMailer, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-37.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, PHPMailer, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-36.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, PHPMailer, MediaElement, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-35.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, PHPMailer, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-34.php Update multiple external library URLs (Class POP3, imgAreaSelect, jQuery/jQuery UI, PHPMailer, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-33.php Remove trailing whitespace and update multiple external library URLs (Class POP3, jQuery/jQuery UI, jQuery Hotkeys, PHPMailer, Thickbox, TinyMCE, etc.).
api.wordpress.org/public_html/core/credits/wp-32.php Update multiple external library URLs including SWFObject and SWFUpload links.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dd32
Copy link
Copy Markdown
Member

dd32 commented May 21, 2026

Claudes review has flagged:

3 endpoints that 404 on the new URL — but they were already 404 on the old http URL too, so the PR is just doing the http→https upgrade and not introducing breakage:

My review:

  • PclZip seems correct url, the site is undergoing updates.
  • TouchPunch is 404 with DEPLOYMENT_NOT_FOUND but https://github.com/furf/jquery-ui-touch-punch still links to that url though.
  • odyniec.net appears to have expired (Sept 2021 registration date) and taken over for other purposes. I'm sadly unable to find any activity from Michał online in the last few years, I would suggest updating this to the GitHub URL: https://github.com/odyniec/imgareaselect

dd32 and others added 2 commits May 21, 2026 03:07
The previous URLs no longer resolve: odyniec.net has expired and
touchpunch.furf.com returns a 404. Use the GitHub project URLs
which are still authoritative, as noted in the PR review.
Copilot AI review requested due to automatic review settings May 21, 2026 03:09
@bazza bazza closed this in 25c9585 May 21, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 40 out of 40 changed files in this pull request and generated no new comments.

@peterwilsoncc peterwilsoncc deleted the remove-credit-redirects branch May 21, 2026 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants