Skip to content

Allow WordPress.org Update URI when slug matches plugin directory#1289

Merged
davidperezgar merged 2 commits intotrunkfrom
1287-relax-update-uri-check
Apr 29, 2026
Merged

Allow WordPress.org Update URI when slug matches plugin directory#1289
davidperezgar merged 2 commits intotrunkfrom
1287-relax-update-uri-check

Conversation

@ernilambar
Copy link
Copy Markdown
Member

Fixes #1287

Relaxes Plugin_Updater_Check so the Update URI header is not flagged when it matches the same URL patterns accepted by the plugin directory API: wordpress.org or w.org, /plugin(s)/…, and the path slug matches the plugin slug. Any other value still reports plugin_updater_detected.

@ernilambar ernilambar marked this pull request as ready for review April 29, 2026 05:57
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 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.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: ernilambar <nilambar@git.wordpress.org>
Co-authored-by: davidperezgar <davidperez@git.wordpress.org>
Co-authored-by: dd32 <dd32@git.wordpress.org>

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

@ernilambar ernilambar requested a review from dd32 April 29, 2026 06:00
@davidperezgar davidperezgar merged commit abba5c0 into trunk Apr 29, 2026
26 checks passed
@davidperezgar davidperezgar deleted the 1287-relax-update-uri-check branch April 29, 2026 15:23
@davidperezgar davidperezgar added this to the 2.0.0 milestone May 2, 2026
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.

Update URI header check too strict

3 participants