fix: Fixed a bug where spreading matchAll() result not working with b…

…uilt files (#1143)

discussion:
https://sendbird.slack.com/archives/G01290GCDCN/p1718676602610299

src/modules/Message/utils/tokens/__tests__/asSafeUrl.spec.ts

@@ -0,0 +1,20 @@
+import { asSafeURL } from '../asSafeURL';
+
+describe('asSafeURL', () => {
+ test('should return the same URL if it is already safe', () => {
+ expect(asSafeURL('http:https://example.com')).toBe('http:https://example.com');
+ expect(asSafeURL('https://example.com')).toBe('https://example.com');
+ });
+
+ test('should return a safe URL if it is not safe', () => {
+ expect(asSafeURL('mailto:[email protected]')).toBe('#');
+ // eslint-disable-next-line no-script-url
+ expect(asSafeURL('javascript:alert(1)')).toBe('#');
+ expect(asSafeURL('javascript%3Aalert%281%29')).toBe('#');
+ expect(asSafeURL('data:text/html;base64,ABCDE==')).toBe('#');
+ });
+
+ test('should append a https:// protocol to the URL if it is missing', () => {
+ expect(asSafeURL('example.com')).toBe('https://example.com');
+ });
+});

src/modules/Message/utils/tokens/tokenize.ts

@@ -116,7 +116,7 @@ export function splitTokensWithMarkdowns(tokens: Token[]): Token[] {
  }
  const rawStr = token.value;
  // @ts-ignore
- const matches = [...rawStr.matchAll(MarkdownRegex)];
+ const matches = Array.from(rawStr.matchAll(MarkdownRegex));
  const allMatches = matches.map((value) => {
  const text = value[0];
  const start = value.index ?? 0;