Comments (3)
Some thoughts:
-
There's no need for an explicit "pre-check". You already have a perfectly valid "test file" - the metadata URL. Let the plugin optimistically use HTTPS the first time it checks for updates. If the request fails, the update checker can set an "SSL doesn't work" flag in the database and use HTTP for future checks.
The only drawback is that users whose server doesn't support SSL won't see any updates for the first
$checkPeriod
hours. I think that's acceptable. -
Automatically downgrading from HTTPS to HTTP is too dangerous to make it the default behaviour. This feature must be opt-in only. I'm thinking something like this:
$updateChecker->enableHttpFallback($enabled, $retryHttpsAfter = 30 * 24 * 3600);
-
The update server already dynamically changes the package URL based on the current protocol (clients using HTTPS get
https://...
URLs). Unless you pass your own URL to theWpup_UpdateServer
constructor, that is.
from wp-update-server.
Hi Yahnis,
thanks a lot for your feedback - havent thought about using the metadata URL as "pre-check" if https is supported. Will give it a try and let you know the results!
best,
Robert
from wp-update-server.
Not an issue anymore, can be closed.
from wp-update-server.
Related Issues (20)
- Issue with the latest wp-update-server version HOT 5
- Critical Error Upon Setup HOT 4
- PHP8 deprecation notices HOT 4
- Sudenly I get: Update failed: You have the latest version of the plugin. HOT 10
- Handling alpha and beta plugin versions HOT 2
- Problem connecting server with checker library HOT 1
- Is it possible to add a list of domains to update? HOT 5
- How about implementing WordPress Code Standards? HOT 4
- Server maintenance questions. HOT 4
- PHP 8.2.7 Deprecated warnings on Server returning metadata (WP_DEBUG true) HOT 3
- Extension for packages list HOT 4
- Need a little help with extends Wpup_UpdateServer HOT 5
- What's going on here? HOT 3
- Can you change it to a remote zip URL? HOT 2
- guidance HOT 1
- Forbidden HOT 13
- Update Packagist HOT 2
- where I have to put example-theme.zip and which class is correct ? HOT 9
- unexpected error HOT 1
- Secure download link error message HOT 1
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 wp-update-server.