Go back to the deprecated url.parse

While deprecated, it's much cleaner to use this when we have relative URLs. It also appears to be a lot faster.

Also, there's an open issue about this in Node.js nodejs/node#12682

routes/appendLocals.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const path = require('path');
+const url = require('url');
 const { generateSri, capitalize } = require('../lib/helpers');
 const { app, files } = require('../config');
 
@@ -58,11 +59,9 @@ function generateSRI(file) {
 function appendLocals(req, res) {
     const siteUrl = getCurrentSiteurl(req);
     const pageUrl = req.originalUrl;
-    // OK, hack-ish way...
-    const pathname = pageUrl.split('?')[0];
-    const canonicalUrl = new URL(pathname, app.siteurl);
+    const canonicalUrl = url.resolve(app.siteurl, url.parse(pageUrl).pathname);
     const theme = getThemeQuery(req);
-    const bodyClass = generateBodyClass(pathname);
+    const bodyClass = generateBodyClass(pageUrl);
 
     const locals = {
         siteUrl,

test/test_helpers.js

@@ -6,6 +6,7 @@ const ENV = process.env;
 ENV.BCDN_HEADERS = ENV.BCDN_HEADERS || 'production';
 
 const assert = require('assert').strict;
+const url = require('url');
 const escapeStringRegexp = require('escape-string-regexp');
 const { htmlEncode } = require('htmlencode');
 const mockDate = require('mockdate');
@@ -141,7 +142,7 @@ function assertPageHeader(txt, res, cb) {
 }
 
 function assertCanonicalUrl(str, res, cb) {
-    const expected = `<link rel="canonical" href="${new URL(str, config.siteurl)}">`;
+    const expected = `<link rel="canonical" href="${url.resolve(config.siteurl, str)}">`;
 
     assert.ok(res.body.includes(expected), 'Expects page to have a canonical URL');
     cb();