Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Download plugins and themes directly from WordPress.org, not via plugin-proxy.php #711

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

adamziel
Copy link
Collaborator

WordPress.org now serves the Access-Control-Allow-* headers necessary to perform a direct fetch() without going through plugin-proxy.php. This commit thus removes the usages of plugin-proxy.php.

The plugin proxy is now only used by the Pull Request previewers. As a follow up, let's consider splitting them to a separate package or even to the playground-tools repo.

Testing Instructions

To test, go to http://localhost:5400/website-server/?theme=pendant&plugin=gutenberg and confirm that both the Pendant theme and the Gutenberg plugins are installed, and also that the zip files are downloaded directly from wordpress.org

cc @akirk

…in-proxy.php

WordPress.org now serves the `Access-Control-Allow-*` headers necessary
to perform a direct `fetch()` without going through plugin-proxy.php.
This commit thus removes the usages of plugin-proxy.php.

The plugin proxy is now only used by the Pull Request previewers. As a
follow up, let's consider splitting them to a separate package or even
to the playground-tools repo.

To test, go to http://localhost:5400/website-server/?theme=pendant&plugin=gutenberg
and confirm that both the Pendant theme and the Gutenberg plugins are
installed, and also that the zip files are downloaded directly from wordpress.org
@adamziel adamziel merged commit 83fab85 into trunk Oct 19, 2023
4 checks passed
@adamziel adamziel deleted the download-plugins-without-proxy branch October 19, 2023 12:36
adamziel added a commit that referenced this pull request Oct 19, 2023
Since Plugin proxy is no longer needed for Playground core (See #711),
let's remove the related .htaccess rewrite.
adamziel added a commit that referenced this pull request Oct 19, 2023
Since Plugin proxy is no longer needed for Playground core (See #711),
let's remove the related .htaccess rewrite.
adamziel added a commit that referenced this pull request Oct 19, 2023
Since Plugin proxy is no longer needed for Playground core (See #711),
let's remove the related .htaccess rewrite.
@akirk akirk mentioned this pull request Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant