Commit Graph

3 Commits

Author SHA1 Message Date
Sebastiaan van Stijn
3c869fb0e3 _layouts: fix redirect layout for external URLs
This allows `redirect_to` redirects to external websites;

    curl localhost:4000/samples/library/adminer/
    <!DOCTYPE html>
    <html lang="en-US">
      <meta charset="utf-8">
      <title>Redirecting&hellip;</title>
      <link rel="canonical" href="https://hub.docker.com/_/adminer/"><meta http-equiv="refresh" content="0; url=https://hub.docker.com/_/adminer/">
      <meta name="robots" content="noindex">
      <h1>Redirecting&hellip;</h1>
      <a href="https://hub.docker.com/_/adminer/">Click here if you are not redirected.</a>
      <script>location="https://hub.docker.com/_/adminer/"</script>
    </html>

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2020-10-21 15:09:14 +02:00
Sebastiaan van Stijn
cafd5a9943 Use full URLs for canonical links
Canonical links are expected to be full URLs, not relative.

For this to work, the Dockerfile had to be updated, because we're stripping
the domain-name from links ("<a href..."), but the script currently also included
"<link rel='canonical' .." tags.

With the change, canonical links are left alone;

These hrefs will be replaced

    echo '<a class=foo href="https://docs.docker.com/foo">hello</a>' | sed -e 's#\(<a[^>]* href="\)https://docs.docker.com/#\1/#g'
    # <a class=foo href="/foo">hello</a>
    echo '<a href="https://docs.docker.com/foo">hello</a>' | sed -e 's#\(<a[^>]* href="\)https://docs.docker.com/#\1/#g'
    # <a href="/foo">hello</a>

But, for example, this one is left alone

    echo '<link rel="canonical" href="https://docs.docker.com/foo/bar" />' | sed -e 's#\(<a[^>]* href="?\)https://docs.docker.com/#\1/#g'
    # <link rel="canonical" href="https://docs.docker.com/foo/bar" />

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2020-10-12 13:09:32 +02:00
Steven Hanna
82896063d5 Fix Liquid Exception (#3316) (#3897)
jekyll-redirect-from v0.12.1 causes "Liquid Exception: invalid byte sequence in US-ASCII in _layouts/redirect.html"

-override default built-in redirect.html with a custom "_layouts/redirect.html" that has the ellipsis HTML fix encoded localy
2017-07-18 13:49:19 -07:00