Merge pull request #6254 from nextcloud/backport/6253/stable4.7

[stable4.7] fix: make call token extraction more robust

src/services/talkService.js

@@ -72,7 +72,7 @@ export async function updateTalkParticipants(eventComponent) {
 		logger.debug('Event\'s conference/location is from another host', url)
 		return
 	}
-	const token = url.match(/\/call\/([a-z0-9]*)$/)[1]
+	const token = extractCallTokenFromUrl(url)
 	if (!token) {
 		logger.debug('URL ' + url + ' contains no call token')
 		return
@@ -148,3 +148,13 @@ export function doesContainTalkLink(text) {
 function generateURLForToken(token = '') {
 	return window.location.protocol + '//' + window.location.host + generateUrl('/call/' + token)
 }
+
+/**
+ * Extract a spreed call token from the given URL
+ *
+ * @param {string} callUrl URL of the spreed call
+ * @return {string|undefined} Matched token or undefined if URL is invalid
+ */
+export function extractCallTokenFromUrl(callUrl) {
+	return callUrl.match(/\/call\/([a-z0-9]*)(\/|#.*)?$/)?.[1] ?? undefined
+}

tests/javascript/unit/services/talkService.test.js

@@ -0,0 +1,21 @@
+/**
+ * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+import { extractCallTokenFromUrl } from '../../../../src/services/talkService'
+
+describe('services/talk test suite', () => {
+	test.each([
+		['https://foo.bar/call/123abc456', '123abc456'],
+		['https://foo.bar/call/123abc456/', '123abc456'],
+		['https://foo.bar/call/123abc456/baz', undefined],
+		['https://foo.bar/call/123abc456#', '123abc456'],
+		['https://foo.bar/call/123abc456#/', '123abc456'],
+		['https://foo.bar/call/123abc456#message_3074226', '123abc456'],
+		['https://foo.bar/baz', undefined],
+		['https://foo.bar/baz/bar', undefined],
+	])('should extract a token from call url %s', (url, expected) => {
+		expect(extractCallTokenFromUrl(url)).toBe(expected)
+	})
+})