rebber: escape link URL

Close #114

packages/rebber/__tests__/__snapshots__/index.js.snap

@@ -9,6 +9,8 @@ a paragraph\\\\footnote{footnoteRawPar inner}
 "
 `;
 
+exports[`link with special characters 1`] = `"\\\\externalLink{foo}{http://example.com?a=b\\\\%c\\\\textasciicircum{}\\\\{\\\\}\\\\#foo}"`;
+
 exports[`mix-1 1`] = `
 "(for convenience,   are replaced with simple single spaces in the tests)
 

packages/rebber/__tests__/index.js

@@ -231,6 +231,17 @@ test('link', () => {
   expect(contents.trim()).toEqual(spec.expected.trim())
 })
 
+test('link with special characters', () => {
+  const {contents} = unified()
+    .use(reParse)
+    .use(rebber)
+    .processSync(dedent`
+      [foo](http://example.com?a=b%c^{}#foo)
+    `)
+
+  expect(contents.trim()).toMatchSnapshot()
+})
+
 test('link-prepend', () => {
   const spec = specs['link-prepend']
 
@@ -277,11 +288,11 @@ test('footnotes', () => {
     .use(rebber, integrationConfig)
     .processSync(dedent`
     # mytitle[^footnoteRef]
-    
+
     [^fotnoteRef]: reference in title
-    
+
     # mytitle[^footnoterawhead inner]
-    
+
     a paragraph[^footnoteRawPar inner]`)
   expect(contents).toMatchSnapshot()
 })

packages/rebber/src/types/link.js

@@ -1,11 +1,12 @@
 /* Dependencies. */
 const has = require('has')
 const all = require('../all')
+const escape = require('../escaper')
 
 /* Expose. */
 module.exports = link
 
-const defaultMacro = (displayedText, link) => `\\externalLink{${displayedText}}{${link}}`
+const defaultMacro = (displayedText, url, title) => `\\externalLink{${displayedText}}{${url}}`
 
 /* Stringify a link `node`.
 */
@@ -14,6 +15,6 @@ function link (ctx, node) {
   const config = ctx.link || {}
   const macro = has(config, 'macro') ? config.macro : defaultMacro
   const prefix = has(config, 'prefix') ? config.prefix : ''
-  const url = node.url.startsWith('/') ? prefix + node.url : node.url
+  const url = escape(node.url.startsWith('/') ? prefix + node.url : node.url)
   return macro(all(ctx, node), url, node.title)
 }