Opened 4 years ago
Closed 4 years ago
#51771 closed defect (bug) (reported-upstream)
Application Passwords Plugin misses 'rest_add_application_passwords_to_index'
Reported by: | pacifika | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | Login and Registration | Keywords: | |
Focuses: | rest-api | Cc: |
Description
After reading the integration guide (https://make.wordpress.org/core/2020/11/05/application-passwords-integration-guide/), I'm excited to use the Application Passwords feature plugin. Thanks for all the hard work to make WP a viable backend for mobile and desktop apps.
However it is missing critical functionality that's present in WordPress 5.6, namely the functionality to detect whether the feature is present. As a result the feature plugin isn't compatible with the integration guide written for it.
For projects that will update to WP 5.6 in the future, but need to use Application Passwords now, the feature plugin is almost perfectly positioned, if it only contained the full feature.
I think it would help adoption of this feature if the feature code written for WP 5.6 was backported into the Application Password feature plugin.
Change History (2)
#2
@
4 years ago
- Milestone Awaiting Review deleted
- Resolution set to reported-upstream
- Status changed from new to closed
Hi @pacifika!
This should be reported in the Application Passwords repo: https://github.com/wordpress/application-passwords
However, I wouldn't recommend developing off of the App Passwords feature plugin. The feature in 5.6 varies greatly in implementation from the featured plugin.
Sorry forgot the crucial info. I'm talking about this function.
https://github.com/WordPress/wordpress-develop/blob/530493396b324f5bed518a494e2843e7fdb020f1/src/wp-includes/rest-api.php#L1099-L1119