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

PHP8 and XML-RPC #86

Open
jjtkalt opened this issue Jan 4, 2025 · 1 comment
Open

PHP8 and XML-RPC #86

jjtkalt opened this issue Jan 4, 2025 · 1 comment
Labels
enhancement New feature or request

Comments

@jjtkalt
Copy link

jjtkalt commented Jan 4, 2025

Heya all! With the directions for W4OS suggesting PHP8 (8.2 currently) but requiring xmlrpc, what is the longer term support plan for this? Specifically, PHP8 dropped the xmlrpc from core (claiming "unused". How... helpful! sigh). Yes, your instructions do mention that you can get it via PECL, but that hasn't been updated or really touched for about two years now, and even that version was beta.

There are other projects out there to support xmlrpc, but the most successful of them has been a class that replaces all of the functions with in-line code.

Regardless, php-xmlrpc is basically not maintained and going away. So, do you have a plan in place to handle that?

GuduleLapointe added a commit that referenced this issue Jan 4, 2025
@GuduleLapointe
Copy link
Owner

GuduleLapointe commented Jan 4, 2025

Hey, thank you for your feedback, it's quite interesting. As xmlrpc is used by both WordPress and OpenSimulator, and libxmlrpc Debian package is still distributed from php repo (although not in core, and with a similar warning), I didn't expect support to be dropped.

I'll surely consider replacing the lib package with a custom library (probably with phpxmlrpc/phpxmlrpc unless I get better advice), but the libxmlrpc functions are used 49 times in 9 different scripts, so it will require extensive testing to make sure it does not break functionalities.

I prepared a new xmlrpc class for reference, we should be able to use it instead without too much changes in the code and strategy.

@GuduleLapointe GuduleLapointe added the enhancement New feature or request label Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants