]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/bugs/trouble_with_base_in_search.mdwn
(no commit message)
[git.ikiwiki.info.git] / doc / bugs / trouble_with_base_in_search.mdwn
1 For security reasons, one of the sites I'm in charge of uses a Reverse Proxy to grab the content from another machine behind our firewall.
2 Let's call the out-facing machine Alfred and the one behind the firewall Betty.
4 For the static pages, everything is fine.  However, when trying to use the search, all the links break.
5 This is because, when Alfred passes the search query on to Betty, the search result has a "base" tag which points to Betty, and all the links to the "found" pages are relative.
6 So we have
8     <base href="Betty.example.com"/>
9     ...
10     <a href="./path/to/found/page/">path/to/found/page</a>
12 This breaks things for anyone on Alfred, because Betty is behind a firewall and they can't get there.
14 What would be better is if it were possible to have a "base" which didn't reference the hostname, and for the "found" links not to be relative.
15 Something like this:
17     <base href="/"/>
18     ...
19     <a href="/path/to/found/page/">path/to/found/page</a>
21 The workaround I've come up with is this.
23 1. Set the "url" in the config to '&nbsp;' (a single space).  It can't be empty because too many things complain if it is.
24 2. Patch the search plugin so that it saves an absolute URL rather than a relative one.
26 Here's a patch:
28     diff --git a/IkiWiki/Plugin/search.pm b/IkiWiki/Plugin/search.pm
29     index 3f0b7c9..26c4d46 100644
30     --- a/IkiWiki/Plugin/search.pm
31     +++ b/IkiWiki/Plugin/search.pm
32     @@ -113,7 +113,7 @@ sub indexhtml (@) {
33             }
34             $sample=~s/\n/ /g;
35     
36     -       my $url=urlto($params{destpage}, "");
37     +       my $url=urlto($params{destpage}, undef);
38             if (defined $pagestate{$params{page}}{meta}{permalink}) {
39                     $url=$pagestate{$params{page}}{meta}{permalink}
40             }
42 It works for me, but it has the odd side-effect of prefixing links with a space.  Fortunately that doesn't seem to break browsers.
43 And I'm sure someone else could come up with something better and more general.
45 --[[KathrynAndersen]]
47 > The `<base href>` is required to be genuinely absolute (HTML 4.01 ยง12.4).
48 > Have you tried setting `url` to the public-facing URL, i.e. with `alfred`
49 > as the hostname? That seems like the cleanest solution to me; if you're
50 > one of the few behind the firewall and you access the site via `betty`
51 > directly, my HTTP vs. HTTPS cleanup in recent versions should mean that
52 > you rarely get redirected to `alfred`, because most URLs are either
53 > relative or "local" (start with '/'). --[[smcv]]
55 >> I did try setting `url` to the "Alfred" machine, but that doesn't seem clean to me at all, since it forces someone to go to Alfred when they started off on Betty.
56 >> Even worse, it prevents me from setting up a test environment on, say, Cassandra, because as soon as one tries to search, one goes to Alfred, then Betty, and not back to Cassandra at all.
57 >> Hardcoded solutions make me nervous.
59 >> I suppose what I would like would be to not need to use a `<base href>` in searching at all.
60 >> --[[KathrynAndersen]]
62 >>> `<base href>` is *not* required to be absolute in HTML5, so when
63 >>> `html5: 1` is used, I've changed it to be host-relative in most cases.
64 >>> I think that at least partially addresses this bug report,
65 >>> particularly if we [[todo/generate HTML5 by default]] like I've suggested.
66 >>>
67 >>> The `<base>` is there so we can avoid having to compute how to
68 >>> get to (the virtual directory containing) the root of the wiki from
69 >>> `ikiwiki.cgi`, which might well be somewhere odd like `/cgi-bin/`.
70 >>> I think there are probably other things that it fixes or simplifies.
71 >>> --[[smcv]]