refactor: use url.from() as Node.js 8 is dropped

hexojs · Apr 21, 2020 · d9f8054 · d9f8054
1 parent c102bba
commit d9f8054
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 9 deletions.
diff --git a/lib/decode_url.js b/lib/decode_url.js
@@ -1,7 +1,6 @@
 'use strict';
 
-const { parse } = require('url');
-const { toUnicode } = require('punycode.js');
+const { parse, format } = require('url');
 
 const safeDecodeURI = str => {
   try {
@@ -18,8 +17,7 @@ const decodeURL = str => {
     // Exit if input is a data url
     if (parsed.origin === 'null') return str;
 
-    // TODO: refactor to `url.format()` once Node 8 is dropped
-    const url = parsed.toString().replace(parsed.hostname, toUnicode(parsed.hostname));
+    const url = format(parsed, { unicode: true });
     return safeDecodeURI(url);
   }
 

diff --git a/lib/encode_url.js b/lib/encode_url.js
@@ -1,7 +1,6 @@
 'use strict';
 
-const { toUnicode } = require('punycode.js');
-const { parse } = require('url');
+const { parse, format } = require('url');
 
 const safeDecodeURI = str => {
   try {
@@ -20,8 +19,7 @@ const encodeURL = str => {
 
     parsed.search = encodeURI(safeDecodeURI(parsed.search));
     // preserve IDN
-    // TODO: refactor to url.format() once Node 8 EOL
-    return parsed.toString().replace(parsed.hostname, toUnicode(parsed.hostname));
+    return format(parsed, { unicode: true });
   }
 
   return encodeURI(safeDecodeURI(str));

diff --git a/package.json b/package.json
@@ -50,7 +50,6 @@
     "highlight.js": "^9.13.1",
     "htmlparser2": "^4.0.0",
     "prismjs": "^1.17.1",
-    "punycode.js": "^2.1.0",
     "strip-indent": "^3.0.0",
     "striptags": "^3.1.1"
   },