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

Look into supporting WP_Filesystem #151

Open
joehoyle opened this issue Mar 1, 2017 · 1 comment
Open

Look into supporting WP_Filesystem #151

joehoyle opened this issue Mar 1, 2017 · 1 comment

Comments

@joehoyle
Copy link
Member

joehoyle commented Mar 1, 2017

Some plugins use the WP_Filesystem API to write to the uploads folder (for unknown reasons) for example: https://github.com/siteorigin/so-widgets-bundle/blob/1.6.5/base/siteorigin-widget.class.php#L628

If easy, it would be good to support this for the s3:// url scheme.

@sc0ttius
Copy link

sc0ttius commented Nov 29, 2017

I think I've run into this issue with Formidable and Visual Composer. The file seems to upload properly to s3, but when building the URL using $wp_filesystem and uploads['basedir'], it uses s3.amazonaws.com/[bucket_name]/ rather than the defined S3_UPLOADS_BUCKET_URL. Is that what you intend to fix with this issue?

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

No branches or pull requests

2 participants