refactor: Use WHATWG URL instead of node url/querystring

apps/meteor/client/lib/e2ee/rocketchat.e2e.ts

@@ -1,6 +1,3 @@
-import QueryString from 'querystring';
-import URL from 'url';
-
 import type { IE2EEMessage, IMessage, IRoom, IUser, IUploadWithUser, Serialized, IE2EEPinnedMessage } from '@rocket.chat/core-typings';
 import { isE2EEMessage, isEncryptedMessageContent } from '@rocket.chat/core-typings';
 import { Emitter } from '@rocket.chat/emitter';
@@ -731,15 +728,18 @@ class E2E extends Emitter {
 					return;
 				}
 
-				const urlObj = URL.parse(url);
-				// if the URL doesn't have query params (doesn't reference message) skip
-				if (!urlObj.query) {
+				// URLs come from regex-matched message content, so they may be malformed; skip those instead of throwing
+				let urlObj: URL;
+				try {
+					urlObj = new URL(url);
+				} catch (error) {
 					return;
 				}
 
-				const { msg: msgId } = QueryString.parse(urlObj.query);
+				const [msgId, ...extraMsgIds] = urlObj.searchParams.getAll('msg');
 
-				if (!msgId || Array.isArray(msgId)) {
+				// skip if the msg param is missing, empty, or duplicated
+				if (!msgId || extraMsgIds.length) {
 					return;
 				}