Opened 3 years ago
Closed 2 years ago
#13328 closed defect (bug) (fixed)
Forced SSL Admin has broken preview links
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.1 |
| Component: | Administration | Version: | 3.0 |
| Severity: | normal | Keywords: | needs-patch |
| Cc: |
Description
If you have forced SSL admin, then your cookies are only for https and the preview link, which uses http, is unauthenticated, producing a 404 response.
Patch uses https scheme for the preview link when it's being used in the admin.
Attachments (2)
Change History (10)
- Keywords needs-patch added; has-patch removed
- Owner set to nacin
- Status changed from new to accepted
This doesn't work for custom post types.
?p=123 does not redirect, it 404s. You need ?p=123&post_type=actual_post_type for it to work.
I was going to add post_type=$post_type, but this also won't work if the permalink is filtered in some way.
I think the safest way is a brute force str_replace for now.
filosofo informs me in IRC that ?p=123 not serving/redirecting is a regression.
Going to investigate that. In the meantime, here's one of those patches I don't want to be associated with.
Actually, this would be a regression: "?p=123 does not redirect, it 404s"
But trunk, using pretty permalinks, redirects a request like that for a draft post to the correct draft url, post_type arg included. So it seems to work (non-SSL).
+1 and Patch still applies clean!