From 862f9d7e11728aa371df9b53c71509e012be574a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ricardo=20Fern=C3=A1ndez=20Serrata?= <76864299+Rudxain@users.noreply.github.com> Date: Sat, 7 May 2022 21:10:43 -0400 Subject: [PATCH] Refactored XOR.js to call `replace` once (#221) * Refactored to make 1 call to `replace` Now it's optimized and refactored to replace the same chars as before and leave other chars intact. The `s` (dot-all) flag is used to reduce conditional branching * Fixed syntax error --- classes/XOR.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/classes/XOR.js b/classes/XOR.js index 05e3637..87287ca 100644 --- a/classes/XOR.js +++ b/classes/XOR.js @@ -1,5 +1,5 @@ module.exports = class XOR { xor(str, key) { return String.fromCodePoint(...str.split('').map((char, i) => char.charCodeAt(0) ^ key.toString().charCodeAt(i % key.toString().length))) } - encrypt(str, key = 37526) { return Buffer.from(this.xor(str, key)).toString('base64').replace(/\//g, '_').replace(/\+/g, '-'); } - decrypt(str, key = 37526) { return this.xor(Buffer.from(str.replace(/\//g, '_').replace(/\+/g, '-'), 'base64').toString(), key) } -} \ No newline at end of file + encrypt(str, key = 37526) { return Buffer.from(this.xor(str, key)).toString('base64').replace(/./gs, c => ({'/': '_', '+': '-'}[c] || c)); } + decrypt(str, key = 37526) { return this.xor(Buffer.from(str.replace(/./gs, c => ({'/': '_', '+': '-'}[c] || c)), 'base64').toString(), key) } +}