Opened 2 years ago
Last modified 2 years ago
#16909 new defect
some Location headers are using relative url's (2)
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Awaiting Review |
| Component: | General | Version: | 3.1 |
| Severity: | normal | Keywords: | |
| Cc: | WordPress@… |
Description
I started this in #14062 and already did some fixes for 3.1.
Just collecting some more and providing patches.
Attachments (5)
Change History (14)
Fix strategy: admin_url -> self_admin_url as we did in first round as well.
comment:2
voyagerfan5761 — 2 years ago
- Cc WordPress@… added
- Keywords has-patch removed
admin_url will return a absolute URL. self_admin_url() is only needed for pages accessible from both the Network admin (or users admin) as well as the normal admin.
Replying to dd32:
admin_url will return a absolute URL. self_admin_url() is only needed for pages
accessible from both the Network admin (or users admin) as well as the normal
admin.
Ah, okay. Had not thought about that, the first patch was mainly a regex search, so I assume most of it does not apply any longer.
what about /wp-admin/network/edit.php and /wp-amin/user/admin.php? Is it the same?
Found it via the options page. A _wp_http_referer is placed in there and later used in options.php to redirect to (/wp-admin/options.php:161). It was no full absolute location.
Better HTTP Redirects now supports WB_DEBUG and displays extensive information about the redirect.
/wp-admin/includes/dashboard.php:127 creates a redirect w/o an absolute URI.

Some leftover