-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Labels
must haveMust be done, high priorityMust be done, high priority
Description
Steps to reproduce:
- Enable and configure the Basic Auth functionality for an environment
- Create an Application Password (this triggers
WP_Application_Passwords::is_in_use()to return true by setting theusing_application_passwordsnetwork setting) - Send a REST API request to the environment with your Basic Auth setting set
Publicly-accessible API endpoints should be accessible; instead, they return a 401.
This is increased priority, as if sending REST API requests from the browser, the 401 returned by Application Passwords causes the browser's internal auth cache to be reset, which requires users to log in again repeatedly.
Basic Auth should take priority here as it's site-wide, but this will mean that Application Passwords can't actually be used in combination. I think that's an acceptable compromise, as regular Require Login can be used in those cases instead, but we should ensure it's documented.
Acceptance criteria:
- Sending a request with Basic Auth headers to a public REST API endpoint (e.g.
/wp-json/) should return a 200 response - Documentation should indicate Basic Auth is not compatible with Application Passwords
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
must haveMust be done, high priorityMust be done, high priority