#51602 closed enhancement (fixed)
Application Passwords: Include siteurl in return data from auth flow.
Reported by: | georgestephanis | Owned by: | TimothyBlynJacobs |
---|---|---|---|
Milestone: | 5.6 | Priority: | normal |
Severity: | normal | Version: | 5.6 |
Component: | Login and Registration | Keywords: | has-patch |
Focuses: | rest-api | Cc: |
Description
Rather than rely on applications to store and connect the site's url with the session the user returns to (which can be fragile with multiple concurrent blogs being connected at once) let's just pass back the url of the site that the credentials are generated for.
Change History (7)
This ticket was mentioned in PR #650 on WordPress/wordpress-develop by georgestephanis.
4 years ago
#1
- Keywords has-patch added
georgestephanis commented on PR #650:
4 years ago
#2
I hesitate to do that as part of the goal is to be agnostic. Given the site url, the client can always determine the rest url, but the client may also want the credentials to use for xmlrpc or other systems like WPGraphQL
#3
@
4 years ago
- Owner set to TimothyBlynJacobs
- Resolution set to fixed
- Status changed from new to closed
In 49291:
TimothyBJacobs commented on PR #650:
4 years ago
#4
Good point, that makes sense.
Merged in 1e85024fb4be72eb6a1244d0c7c09590ed7172a2.
username
touser_login
-- we've been doing the latter in practice, but some code still returned the former.site_url
parameter to the return data so clients have confirmation of what url they can use the returned credentials with.Trac ticket: https://core.trac.wordpress.org/ticket/51602#ticket