Fix support for URLs containing a query string

Closes GH-37.

Co-authored-by: Titus Wormer <tituswormer@gmail.com>

index.js

@@ -31,6 +31,7 @@ var fileRuleId = 'missing-file'
 cliCompleter.pluginId = sourceId
 
 var parse = url.parse
+var format = url.format
 
 var viewPaths = {
   github: 'blob',
@@ -296,6 +297,10 @@ function gatherReferences(file, tree, info, fileSet) {
 
     uri = parse(link)
 
+    // Drop `?search`
+    uri.search = ''
+    link = format(uri)
+
     if (!fileSet && (uri.hostname || uri.pathname)) {
       return
     }

test/fixtures/query-params.md

@@ -0,0 +1,7 @@
+# Query params
+
+Link to relative heading ![link](?foo=bar#query-params)
+
+Link to an ![image](./examples/image.jpg?foo=bar)
+
+And a file [link](./examples/github.md?foo=bar).

test/index.js

@@ -524,5 +524,25 @@ test('remark-validate-links', function(t) {
     }, st.error)
   })
 
+  t.test('should support query parameters', function(st) {
+    st.plan(1)
+
+    execa(bin, [
+      '--no-config',
+      '--no-ignore',
+      '--use',
+      '../..=repository:"wooorm/test"',
+      '--use',
+      '../sort',
+      'query-params.md'
+    ]).then(function(result) {
+      st.equal(
+        strip(result.stderr),
+        'query-params.md: no issues found',
+        'should report'
+      )
+    }, st.error)
+  })
+
   t.end()
 })