Comments (9)
Absolutely correct, but will require substantial refactoring, esp. to make sure the links to autoptimized CSS/ JS result in the cached CSS/ JS being fetched from FS/ GAE cloud storage bucket.
Giving this complexity and given the fact this isn't a feature anyone has ever requested (except you, here, now), I'll consider this of low priority.
frank
from autoptimize.
I did quick web-search on "wp_filesystem google app engine" I find no (immediate) confirmation that wp_filesystem automagically solves any filesystem issues. From what I read on https://cloud.google.com/appengine/docs/php/googlestorage/ the way to write files is by using the:
Google Cloud Storage stream wrapper is built in to the run time, and is used when you supply a file name starting with gs://.
While WP's filesystem API tries to write using local filesystem, ftp or ssh.
So although I understand GAE has it's challenges (and AO won't work on there for now), I don't think wp_filesystem is the solution?
frank
from autoptimize.
I think this plugin provides the WP FileSystem API implementation:
https://wordpress.org/plugins/google-app-engine/
from autoptimize.
And WP's filesystem API would support any custom filesytem.
BTW I am planning to use https://github.com/kasparsd/minit/tree/feature/restructure
from autoptimize.
I think this plugin provides the WP FileSystem API implementation:
https://wordpress.org/plugins/google-app-engine/
I checked the code, and no, it does not extend wp_filesystem, it "only" has code to do mail, http-requests and file-uploads using GAE.
And WP's filesystem API would support any custom filesytem.
Had a look at the code (again) and It does not; it "only" supports direct file access, ftp (+ variation for ftp) or ssh, all of which won't work on GAE. This is the very reason why on GAE you actually have to update wp core, theme & plugins on your local machine and then deploy to GAE (as described on this page, under "8.Prepare local site for upload";
Run all available updates, you will not be able to do this when the site is uploaded. To install plugins and updates you have to do it locally and then reupload. Therefor it’s a good idea to keep your local environment to be able to take care about this later on.
So the conclusion; wp_filesystem does not help us to make AO work on GAE. This does not mean using wp_filesystem would be entirely useless, but there's no clear, outspoken advantage in doing so either.
frank
from autoptimize.
Thank you.
but a role of thumb is that caching and minifying plugins won’t work.
😄
from autoptimize.
but a role of thumb is that caching and minifying plugins won’t work.
yeah ... I think I could make it work (writing the optimized files using Google Cloud Storage stream wrapper and injecting links to optimized code of the type wp-content/plugins/autoptimize/gae.php?id=xyz.css with gae.php using the stream wrapper to read the file), but it would introduce some complexity that -given the low amount of WP on GAE- I feel is not really worth it.
anyway, thanks for challenging me, I learned from it :-)
frank
from autoptimize.
I think when a project uses GAE resources are optimized during the build process.
from autoptimize.
Doesn't look like it when checking the HTML source on https://gae-php-tips.appspot.com/...
There used to be mod_pagespeed integration in GAE, but that was turned off late 2015.
from autoptimize.
Related Issues (20)
- Make compatibility configurable HOT 8
- Compatible wih WP 6 and PHP 8.1 HOT 1
- Keep the code base clean HOT 3
- Undefined offset autoptimizeStyles.php HOT 3
- autoptimizecriticalcsscron PHP array key warnings HOT 5
- Avif support broken HOT 6
- Minified JS broken with private class identifiers HOT 4
- Disable emoji dns-prefetch in ClassicPress HOT 1
- PHP8 Warning HOT 1
- Optimized JS and inline script order HOT 3
- Problem with media="print" HOT 3
- Inline "background" not replaced with shortpixel HOT 5
- Security risk reported by Kaspersky HOT 6
- Proposal: stronger default settings HOT 9
- Optimised Google Fonts still blocking for 1.3s according to PageSpeed HOT 2
- Add: option to enable WebP as default output format HOT 1
- PHP Warning in cronjob: Trying to access array offset on value of type int in autoptimizeCacheChecker.php HOT 3
- Creation of dynamic property in autoptimizeProTab.php HOT 4
- Proposal: add Plugin Check workflow HOT 6
- Autoptimize deletes all subfolders in /cache/ HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from autoptimize.