aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoey Hess <joey@kitenet.net>2010-09-26 23:01:01 -0400
committerJoey Hess <joey@kitenet.net>2010-09-26 23:01:01 -0400
commite7b772e65b5cedb6773a6ecc52cde6fb0bacdd79 (patch)
tree6def5fbc7f571c4ee2348a78966992edd0f50799
parent1334695f4ca8a08ea94c176f5cc4c1fafa7aa31c (diff)
downloadikiwiki-e7b772e65b5cedb6773a6ecc52cde6fb0bacdd79.tar
ikiwiki-e7b772e65b5cedb6773a6ecc52cde6fb0bacdd79.tar.gz
review
-rw-r--r--doc/todo/want_to_avoid_ikiwiki_using_http_or_https_in_urls_to_allow_serving_both.mdwn19
1 files changed, 19 insertions, 0 deletions
diff --git a/doc/todo/want_to_avoid_ikiwiki_using_http_or_https_in_urls_to_allow_serving_both.mdwn b/doc/todo/want_to_avoid_ikiwiki_using_http_or_https_in_urls_to_allow_serving_both.mdwn
index 112112284..1547c39eb 100644
--- a/doc/todo/want_to_avoid_ikiwiki_using_http_or_https_in_urls_to_allow_serving_both.mdwn
+++ b/doc/todo/want_to_avoid_ikiwiki_using_http_or_https_in_urls_to_allow_serving_both.mdwn
@@ -82,3 +82,22 @@ you don't like my approach:
than a URL for the user's browser, so I left those alone)
--[[smcv]]
+
+> The justification for your patch seems to be wanting to use a different
+> domain, like secure.foo.com, for https? Can you really not just configure
+> both url and cgiurl to use `https://secure.foo.com/...` and rely on
+> relative links to keep users of `http://insecure.foo.com/` on http until
+> they need to use the cgi?
+>
+> I'm unconvinced.
+>
+> `Ikiwiki::baseurl()."foo"` just seems to be asking for trouble,
+> ie being accidentially written as `IkiWiki::baseurl("foo")`,
+> which will fail when foo is not a page, but some file.
+>
+> I see multiple places (inline.pm, meta.pm, poll.pm, recentchanges.pm)
+> where it will now put the https url into a static page if the build
+> happens to be done by the cgi accessed via https, but not otherwise.
+> I would rather not have to audit for such problems going forward.
+>
+> --[[Joey]]