Fix(signaling): ignore ECONNABORTED errors to prevent conversation not found

[Paulcor35]

…t found

☑️ Resolves

Prevents the "Conversation not found" error page when a temporary network interruption (ECONNABORTED) occurs.

Previously, if a polling request was aborted (e.g., refreshing the page or unstable network), the user was kicked out of the conversation. This patch ignores ECONNABORTED errors just like we ignore isCancel.

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

Screenshot before

Screenshot after

No UI changes. The conversation loads normally instead of showing the "Conversation not found" error.

🚧 Tasks

• Add ECONNABORTED to ignored errors in signaling.js

🏁 Checklist

• 🌏 Tested with different browsers / clients:
  • Chromium (Chrome / Edge / Opera / Brave)
  • Firefox
  • Safari
  • Talk Desktop
  • Integrations with Files sidebar and other apps
  • Not risky to browser differences / client
• 🖌️ Design was reviewed, approved or inspired by the design team
• ⛑️ Tests are included or not possible
• 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

---

[Antreesy]

Otherwise a good addition. Can reproduce locally with small patch (axios aborts request with this code when timeout, and in other cases) + Internal signaling

--- a/src/services/signalingService.js	(revision 972b65935d05780ec33a7337a4c28aab0d6fc9e7)
+++ b/src/services/signalingService.js	(date 1771412086368)
@@ -26,7 +26,7 @@
  * @param options
  */
 async function pullSignalingMessages(token, options) {
-	return axios.get(generateOcsUrl('apps/spreed/api/v3/signaling/{token}', { token }), options)
+	return axios.get(generateOcsUrl('apps/spreed/api/v3/signaling/{token}', { token }), { timeout: 1 })
 }
 

[Antreesy]

Looping @danxuliu for a second opinion