Merge pull request #2596 from oxen-io/fix-sogs-room-with-uppercase

fix: do not lowercase roomId before joining sogs room

ts/session/apis/open_group_api/opengroupV2/ApiUtil.ts

@@ -1,4 +1,4 @@
-import _, { compact, flatten, isString } from 'lodash';
+import _, { clone, compact, flatten, isString } from 'lodash';
 import { allowOnlyOneAtATime } from '../../../utils/Promise';
 import {
   updateDefaultRooms,
@@ -80,7 +80,7 @@ export function hasExistingOpenGroup(server: string, roomId: string) {
     return false;
   }
 
-  const serverNotLowerCased = server;
+  const serverNotLowerCased = clone(server);
   const serverLowerCase = serverNotLowerCased.toLowerCase();
 
   let serverUrl: URL | undefined;

ts/session/apis/open_group_api/opengroupV2/JoinOpenGroupV2.ts

@@ -22,14 +22,14 @@ import { getOpenGroupManager } from './OpenGroupManagerV2';
 // 143.198.213.255:80/main?public_key=658d29b91892a2389505596b135e76a53db6e11d613a51dbd3d0816adffb231c
 
 export function parseOpenGroupV2(urlWithPubkey: string): OpenGroupV2Room | undefined {
-  const lowerCased = urlWithPubkey.trim().toLowerCase();
+  const trimmed = urlWithPubkey.trim();
   try {
-    if (!openGroupV2CompleteURLRegex.test(lowerCased)) {
+    if (!openGroupV2CompleteURLRegex.test(trimmed)) {
       throw new Error('regex fail');
     }
 
     // prefix the URL if it does not have a prefix
-    const prefixedUrl = prefixify(lowerCased);
+    const prefixedUrl = prefixify(trimmed);
     // new URL fails if the protocol is not explicit
     const url = new URL(prefixedUrl);
 
@@ -43,7 +43,7 @@ export function parseOpenGroupV2(urlWithPubkey: string): OpenGroupV2Room | undef
     };
     return room;
   } catch (e) {
-    window?.log?.error('Invalid Opengroup v2 join URL:', lowerCased, e);
+    window?.log?.error('Invalid Opengroup v2 join URL:', trimmed, e);
   }
   return undefined;
 }
@@ -62,8 +62,8 @@ async function joinOpenGroupV2(room: OpenGroupV2Room, fromConfigMessage: boolean
     return;
   }
 
-  const serverUrl = room.serverUrl.toLowerCase();
-  const roomId = room.roomId.toLowerCase();
+  const serverUrl = room.serverUrl;
+  const roomId = room.roomId;
   const publicKey = room.serverPublicKey.toLowerCase();
   const prefixedServer = prefixify(serverUrl);