Add support for <base> element

wooorm · wooorm · commit cf9223051d09 · 2018-06-23T13:06:16.000+02:00
Closes GH-30.
diff --git a/index.js b/index.js
@@ -10,6 +10,9 @@ var handlers = require('./lib/handlers')
 function toMDAST(tree, options) {
   var settings = options || {}
 
+  h.baseFound = false
+  h.frozenBaseURL = null
+
   h.handlers = xtend(handlers, settings.handlers || {})
   h.augment = augment
   h.document = settings.document
diff --git a/lib/handlers/base.js b/lib/handlers/base.js
@@ -0,0 +1,10 @@
+'use strict'
+
+module.exports = base
+
+function base(h, node) {
+  if (!h.baseFound) {
+    h.frozenBaseURL = node.properties.href || null
+    h.baseFound = true
+  }
+}
diff --git a/lib/handlers/image.js b/lib/handlers/image.js
@@ -2,9 +2,11 @@
 
 module.exports = image
 
+var resolve = require('../util/resolve')
+
 function image(h, node) {
   var props = {
-    url: node.properties.src || '',
+    url: resolve(h, node.properties.src),
     title: node.properties.title || null,
     alt: node.properties.alt || null
   }
diff --git a/lib/handlers/index.js b/lib/handlers/index.js
@@ -19,7 +19,6 @@ exports.command = ignore
 exports.content = ignore
 exports.datalist = ignore
 exports.dialog = ignore
-exports.head = ignore
 exports.input = ignore
 exports.keygen = ignore
 exports.link = ignore
@@ -28,6 +27,7 @@ exports.math = ignore
 exports.menu = ignore
 exports.menuitem = ignore
 exports.meta = ignore
+exports.noscript = ignore
 exports.optgroup = ignore
 exports.option = ignore
 exports.script = ignore
@@ -79,6 +79,7 @@ exports.main = wrapped
 exports.nav = wrapped
 exports.section = wrapped
 
+exports.base = require('./base')
 exports.ol = exports.ul = exports.dir = require('./list')
 exports.table = require('./table')
 exports.tr = require('./table-row')
diff --git a/lib/handlers/link.js b/lib/handlers/link.js
@@ -3,10 +3,11 @@
 module.exports = link
 
 var all = require('../all')
+var resolve = require('../util/resolve')
 
 function link(h, node) {
   var props = {
-    url: node.properties.href || '',
+    url: resolve(h, node.properties.href),
     title: node.properties.title || null
   }
 
diff --git a/lib/util/resolve.js b/lib/util/resolve.js
@@ -0,0 +1,11 @@
+'use strict'
+
+module.exports = resolve
+
+function resolve(h, url) {
+  if (url === null || url === undefined) {
+    return ''
+  }
+
+  return h.frozenBaseURL ? String(new URL(url, h.frozenBaseURL)) : url
+}
diff --git a/test/fixtures/base-invalid/index.html b/test/fixtures/base-invalid/index.html
@@ -0,0 +1,14 @@
+<html>
+  <head>
+    <title>Document title</title>
+    <base target="_blank">
+    <base href="http://some-other-website.com">
+  </head>
+  <body>
+    <p><a href="http://some-other-url.com/relative.html">example</a></p>
+    <p><a href="relative.html">example</a></p>
+    <p><a href="./relative.html">example</a></p>
+    <p><a href="../relative.html">example</a></p>
+    <p><a>example</a></p>
+  </body>
+</html>
diff --git a/test/fixtures/base-invalid/index.md b/test/fixtures/base-invalid/index.md
@@ -0,0 +1,9 @@
+[example](http://some-other-url.com/relative.html)
+
+[example](relative.html)
+
+[example](./relative.html)
+
+[example](../relative.html)
+
+[example](<>)
diff --git a/test/fixtures/base/index.html b/test/fixtures/base/index.html
@@ -0,0 +1,14 @@
+<html>
+  <head>
+    <title>Document title</title>
+    <base href="http://example.com">
+    <base href="http://some-other-website.com">
+  </head>
+  <body>
+    <p><a href="http://some-other-url.com/relative.html">example</a></p>
+    <p><a href="relative.html">example</a></p>
+    <p><a href="./relative.html">example</a></p>
+    <p><a href="../relative.html">example</a></p>
+    <p><a>example</a></p>
+  </body>
+</html>
diff --git a/test/fixtures/base/index.md b/test/fixtures/base/index.md
@@ -0,0 +1,9 @@
+[example](http://some-other-url.com/relative.html)
+
+[example](http://example.com/relative.html)
+
+[example](http://example.com/relative.html)
+
+[example](http://example.com/relative.html)
+
+[example](<>)
