Merge pull request #734 from GetStream/fix/uppercase-url-scheme-linkification

fix(message): recognize uppercase URL schemes

Author: szuperaz

projects/stream-chat-angular/src/lib/message-text/message-text.component.spec.ts

@@ -302,6 +302,42 @@ describe('MessageTextComponent', () => {
     );
   });
 
+  it('should not double-prefix URLs with uppercase or mixed-case schemes', () => {
+    component.message = {
+      text: 'This is a message with a link HTTPS://example.com',
+    } as any as StreamMessage;
+    component.ngOnChanges({ message: {} as SimpleChange });
+
+    expect(component.messageTextParts![0].content).toContain(
+      '<a href="HTTPS://example.com" target="_blank" rel="nofollow">HTTPS://example.com</a>'
+    );
+    expect(component.messageTextParts![0].content).not.toContain(
+      'https://HTTPS://example.com'
+    );
+
+    component.message.text =
+      'This is a message with a link Https://example.com';
+    component.ngOnChanges({ message: {} as SimpleChange });
+
+    expect(component.messageTextParts![0].content).toContain(
+      '<a href="Https://example.com" target="_blank" rel="nofollow">Https://example.com</a>'
+    );
+
+    component.message.text = 'This is a message with a link FTP://example.com';
+    component.ngOnChanges({ message: {} as SimpleChange });
+
+    expect(component.messageTextParts![0].content).toContain(
+      '<a href="FTP://example.com" target="_blank" rel="nofollow">FTP://example.com</a>'
+    );
+
+    component.message.text = 'This is a message with a link example.com';
+    component.ngOnChanges({ message: {} as SimpleChange });
+
+    expect(component.messageTextParts![0].content).toContain(
+      '<a href="https://example.com" target="_blank" rel="nofollow">example.com</a>'
+    );
+  });
+
   it('should replace URL links inside text content - custom link renderer', () => {
     const service = TestBed.inject(MessageService);
     service.customLinkRenderer = (url) =>

projects/stream-chat-angular/src/lib/message-text/message-text.component.ts

@@ -157,11 +157,7 @@ export class MessageTextComponent implements OnChanges {
         return this.messageService.customLinkRenderer(match);
       } else {
         let href = match;
-        if (
-          !href.startsWith('http') &&
-          !href.startsWith('ftp') &&
-          !href.startsWith('file')
-        ) {
+        if (!/^(?:https?|ftp|file):\/\//i.test(href)) {
           href = `https://${match}`;
         }
         return `<a href="${href}" target="_blank" rel="nofollow">${match}</a>`;