Opened 15 years ago
Closed 14 years ago
#13328 closed defect (bug) (fixed)
Forced SSL Admin has broken preview links
Reported by: | filosofo | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.1 | Priority: | normal |
Severity: | normal | Version: | 3.0 |
Component: | Administration | Keywords: | needs-patch |
Focuses: | 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)
#4
@
14 years ago
- 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.
#5
@
14 years ago
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.
#6
@
14 years ago
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.
+1 and Patch still applies clean!