aboutsummaryrefslogtreecommitdiff
path: root/doc/bugs/preview_base_url_should_be_absolute.mdwn
diff options
context:
space:
mode:
authorSimon McVittie <smcv@debian.org>2011-01-05 12:17:17 +0000
committerSimon McVittie <smcv@debian.org>2011-01-05 12:17:17 +0000
commit3d893db65f8109ead4868c7e692277b05fb0ec20 (patch)
treeabd1141ec226468775347e0910ed4efa46b95c31 /doc/bugs/preview_base_url_should_be_absolute.mdwn
parentb79e993d8654f0197f6d28a0eb4c0566c61ebf8b (diff)
downloadikiwiki-3d893db65f8109ead4868c7e692277b05fb0ec20.tar
ikiwiki-3d893db65f8109ead4868c7e692277b05fb0ec20.tar.gz
respond
Diffstat (limited to 'doc/bugs/preview_base_url_should_be_absolute.mdwn')
-rw-r--r--doc/bugs/preview_base_url_should_be_absolute.mdwn17
1 files changed, 17 insertions, 0 deletions
diff --git a/doc/bugs/preview_base_url_should_be_absolute.mdwn b/doc/bugs/preview_base_url_should_be_absolute.mdwn
index b72831b16..fb9a05951 100644
--- a/doc/bugs/preview_base_url_should_be_absolute.mdwn
+++ b/doc/bugs/preview_base_url_should_be_absolute.mdwn
@@ -3,3 +3,20 @@ absolute, which can break the preview function in some circumstances
(with e.g. images not showing). The trivial [[patch]] that fixes
it can be found [[here|http://sprunge.us/EPHT]] as well as on [[my
git|http://git.oblomov.eu/ikiwiki]].
+
+> That patch does mean that if you're accessing the CGI via HTTPS but your
+> $config{url} and $config{cgiurl} are HTTP, you'll get preview images loaded
+> via HTTP, causing the browser to complain. See
+> [[todo/want_to_avoid_ikiwiki_using_http_or_https_in_urls_to_allow_serving_both]]
+> for background.
+>
+> Perhaps the CGI could form its `<base>` URL by using
+> `URI->new_abs(urlto(...), $cgi->url)` instead?
+>
+> You'd also need to change `IkiWiki/Wrapper.pm` to pass at least the
+> SERVER_NAME and SERVER_PORT through the environment, probably.
+>
+> Joey's last comment on
+> [[todo/want_to_avoid_ikiwiki_using_http_or_https_in_urls_to_allow_serving_both]]
+> suggests that this might already work, but I'm not quite sure how - I'd
+> expect it to need more environment variables? --[[smcv]]