Skip to content

Add a few more plugins that blocks XML-RPC#1135

Closed
crazytonyli wants to merge 1 commit intotrunkfrom
xmlrpc/add-more-plugins
Closed

Add a few more plugins that blocks XML-RPC#1135
crazytonyli wants to merge 1 commit intotrunkfrom
xmlrpc/add-more-plugins

Conversation

@crazytonyli
Copy link
Copy Markdown
Contributor

Relates to wordpress-mobile/WordPress-iOS#25183

The new plugins and new xmlrpc_blocking_plugins API will be used in the app to show what plugins may blocked the site's XML-RPC endpoint.

@crazytonyli crazytonyli requested a review from jkmassel February 2, 2026 01:13
@crazytonyli crazytonyli force-pushed the xmlrpc/add-more-plugins branch from e4c9726 to 60e1034 Compare February 2, 2026 02:35
Copy link
Copy Markdown
Contributor

@jkmassel jkmassel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure how this should work – the application blocking plugins can be detected by fetching the JSON API root, but these XMLRPC ones would only be detectable after the user is authenticated, right?

@crazytonyli
Copy link
Copy Markdown
Contributor Author

but these XMLRPC ones would only be detectable after the user is authenticated, right?

Yes, I plan to show them in error messages, which will be built on top of wordpress-mobile/WordPress-iOS#25183

@jkmassel
Copy link
Copy Markdown
Contributor

jkmassel commented Feb 2, 2026

Yes, I plan to show them in error messages

This is a separate type of error though – it's not a KnownAuthenticationBlockingPlugin. Can we introduce a different type and list for this to avoid making KnownAuthenticationBlockingPlugin more complex?

@crazytonyli
Copy link
Copy Markdown
Contributor Author

Sure. I saw there is an xmlrpc function there, I thought I'd build on that.

But after a second look in the error message UI in the app, I think I'll stick with a general message for now.

@crazytonyli crazytonyli closed this Feb 2, 2026
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

Successfully merging this pull request may close these issues.

2 participants