@ethora/chat-component-rn 26.10.1 → 26.10.3

package/src/hooks/useLogout.tsx

@@ -50,7 +50,18 @@ const logoutService = {
    * AFTER all the disk work has actually landed.
    */
   performLogout: async (): Promise<void> => {
-    // 0. Flush lastViewedTimestamp to the server's private store BEFORE
+    // 0. Push. Release the device-token registration on the backend
+    //    FIRST, while the access token is still valid (the token itself
+    //    is kept so the next login re-registers it). It runs alongside
+    //    the private-store flush below instead of after it: the flush
+    //    can take up to 2s, and a host with a logout time budget could
+    //    cut us off before the release went out, leaving the device
+    //    receiving pushes for the logged-out user.
+    const pushRelease = releasePushRegistration().catch((e) => {
+      console.warn('logoutService: push release failed', e);
+    });
+
+    // 0a. Flush lastViewedTimestamp to the server's private store BEFORE
     //    we close XMPP. Only rooms with no outstanding unread (plus the
     //    visible room) get their entries updated — rooms with unread
     //    keep their old marker so the next login still surfaces those
@@ -95,15 +106,9 @@ const logoutService = {
       /* non-fatal */
     }
 
-    // 2. Push. Release the device-token registration on the backend
-    //    while the access token is still valid (the token itself is kept
-    //    so the next login re-registers it), then clear the local MucSub
-    //    room cache.
-    try {
-      await releasePushRegistration();
-    } catch (e) {
-      console.warn('logoutService: push release failed', e);
-    }
+    // 2. Push. Wait for the release started in step 0, then clear the
+    //    local MucSub room cache.
+    await pushRelease;
     try {
       await pushSubscriptionService.reset();
     } catch (e) {

package/src/i18n/strings.ts

@@ -128,6 +128,8 @@ const en: UiStringTable = {
   'call.error.deviceInUse': 'Your camera or microphone is already in use by another app.',
   'call.error.deviceGeneric': 'Could not access your camera or microphone.',
   'call.error.noAnswerTimeout': 'No answer, the call timed out',
+  'call.peerMuted': '{name} is muted',
+  'call.selfMuted': 'Your microphone is muted',
   'action.send': 'Send',
   'action.cancel': 'Cancel',
   'action.save': 'Save',
@@ -138,6 +140,7 @@ const en: UiStringTable = {
   'action.mute': 'Mute',
   'action.unmute': 'Unmute',
   'toast.muteFailed': 'Could not update notifications for this chat',
+  'toast.leaveFailed': 'Could not leave this chat',
   'action.create': 'Create',
   'action.submit': 'Submit',
   'action.translate': 'Translate',
@@ -505,6 +508,8 @@ const fr: UiStringTable = {
   'call.error.deviceInUse': 'Votre caméra ou microphone est déjà utilisé par une autre application.',
   'call.error.deviceGeneric': 'Impossible d\'accéder à votre caméra ou microphone.',
   'call.error.noAnswerTimeout': 'Pas de réponse, l\'appel a expiré',
+  'call.peerMuted': '{name} a coupé son micro',
+  'call.selfMuted': 'Votre micro est coupé',
   'action.send': 'Envoyer',
   'action.cancel': 'Annuler',
   'action.save': 'Enregistrer',
@@ -515,6 +520,7 @@ const fr: UiStringTable = {
   'action.mute': 'Muet',
   'action.unmute': 'Réactiver',
   'toast.muteFailed': 'Impossible de modifier les notifications de ce chat',
+  'toast.leaveFailed': 'Impossible de quitter ce chat',
   'action.create': 'Créer',
   'action.submit': 'Soumettre',
   'action.translate': 'Traduire',
@@ -882,6 +888,8 @@ const es: UiStringTable = {
   'call.error.deviceInUse': 'Tu cámara o micrófono ya está siendo usado por otra aplicación.',
   'call.error.deviceGeneric': 'No se pudo acceder a tu cámara o micrófono.',
   'call.error.noAnswerTimeout': 'Sin respuesta, la llamada expiró',
+  'call.peerMuted': '{name} tiene el micrófono silenciado',
+  'call.selfMuted': 'Tu micrófono está silenciado',
   'action.send': 'Enviar',
   'action.cancel': 'Cancelar',
   'action.save': 'Guardar',
@@ -892,6 +900,7 @@ const es: UiStringTable = {
   'action.mute': 'Silenciar',
   'action.unmute': 'Activar',
   'toast.muteFailed': 'No se pudieron cambiar las notificaciones de este chat',
+  'toast.leaveFailed': 'No se pudo salir de este chat',
   'action.create': 'Crear',
   'action.submit': 'Enviar',
   'action.translate': 'Traducir',
@@ -1259,6 +1268,8 @@ const pt: UiStringTable = {
   'call.error.deviceInUse': 'A sua câmara ou microfone já está a ser utilizado por outra aplicação.',
   'call.error.deviceGeneric': 'Não foi possível aceder à câmara ou microfone.',
   'call.error.noAnswerTimeout': 'Sem resposta, a chamada expirou',
+  'call.peerMuted': '{name} está com o microfone desativado',
+  'call.selfMuted': 'Seu microfone está desativado',
   'action.send': 'Enviar',
   'action.cancel': 'Cancelar',
   'action.save': 'Guardar',
@@ -1269,6 +1280,7 @@ const pt: UiStringTable = {
   'action.mute': 'Silenciar',
   'action.unmute': 'Ativar som',
   'toast.muteFailed': 'Could not update notifications for this chat',
+  'toast.leaveFailed': 'Não foi possível sair deste chat',
   'action.create': 'Criar',
   'action.submit': 'Enviar',
   'action.translate': 'Traduzir',
@@ -1636,6 +1648,8 @@ const ht: UiStringTable = {
   'call.error.deviceInUse': 'Kamera oswa mikwofòn ou deja ap itilize pa yon lòt app.',
   'call.error.deviceGeneric': 'Nou pa t kapab jwenn aksè nan kamera oswa mikwofòn ou.',
   'call.error.noAnswerTimeout': 'Pa gen repons, apèl la fini',
+  'call.peerMuted': '{name} koupe mikwo li',
+  'call.selfMuted': 'Mikwo ou koupe',
   'action.send': 'Voye',
   'action.cancel': 'Anile',
   'action.save': 'Anrejistre',
@@ -1646,6 +1660,7 @@ const ht: UiStringTable = {
   'action.mute': 'Mete an silans',
   'action.unmute': 'Retire silans',
   'toast.muteFailed': 'Could not update notifications for this chat',
+  'toast.leaveFailed': 'Pa t kapab kite chat sa a',
   'action.create': 'Kreye',
   'action.submit': 'Soumèt',
   'action.translate': 'Tradui',
@@ -2013,6 +2028,8 @@ const zh: UiStringTable = {
   'call.error.deviceInUse': '您的摄像头或麦克风正被其他应用占用。',
   'call.error.deviceGeneric': '无法访问您的摄像头或麦克风。',
   'call.error.noAnswerTimeout': '无人接听，通话已超时',
+  'call.peerMuted': '{name} 已静音',
+  'call.selfMuted': '你的麦克风已静音',
   'action.send': '发送',
   'action.cancel': '取消',
   'action.save': '保存',
@@ -2023,6 +2040,7 @@ const zh: UiStringTable = {
   'action.mute': '静音',
   'action.unmute': '取消静音',
   'toast.muteFailed': 'Could not update notifications for this chat',
+  'toast.leaveFailed': '无法退出此聊天',
   'action.create': '创建',
   'action.submit': '提交',
   'action.translate': '翻译',

package/src/networking/api-requests/rooms.api.ts

@@ -1,4 +1,5 @@
 import { ApiRoom, DeleteRoomMember, PostAddRoomMember, PostReportRoom, PostRoom, RoomMember } from '../../types/models/room.model';
+import { uniqueMembers } from '../../helpers/uniqueMembers';
 import { store } from '../../roomStore';
 import { addRoom, mergeUsersSet, updateRoom } from '../../roomStore/roomsSlice';
 import { IRoom } from '../../types/types';
@@ -118,7 +119,7 @@ export function ownAppId(): string {
 
 const toRoomMembers = (members: any[] | undefined): RoomMember[] => {
   const known = (store.getState().rooms?.usersSet || {}) as Record<string, RoomMember>;
-  return (members || []).map((m: any) => ({
+  return uniqueMembers(members).map((m: any) => ({
     firstName: m.firstName,
     lastName: m.lastName,
     profileImage:
@@ -495,6 +496,32 @@ export async function deleteRoomMember(data: DeleteRoomMember) {
   }
 }
 
+/**
+ * Take the signed-in user out of a chat on the server (self-removal via the
+ * same endpoint that removes other members). The unavailable presence alone
+ * only leaves the MUC for this session: the backend keeps the user in
+ * `members` and the next /chats/my brings the chat back. A 404 means the
+ * server already has us out, which is what we wanted.
+ */
+export async function leaveRoom(chatName: string): Promise<void> {
+  const me = (store.getState().chatSettingStore?.user?.xmppUsername || '').split('@')[0];
+  if (!chatName || !me) {
+    throw new Error('Cannot leave the chat: no chat name or user');
+  }
+  const token = store.getState().chatSettingStore.user.token || '';
+  invalidateRoomsCache();
+  try {
+    await http.delete('/v1/chats/users-access', {
+      headers: { Authorization: token },
+      data: { chatName, members: [me] },
+    });
+  } catch (error: any) {
+    if (error?.response?.status !== 404) {throw error;}
+  } finally {
+    invalidateRoomsCache();
+  }
+}
+
 export async function deleteRoom(name: string) {
   const token = store.getState().chatSettingStore.user.token || '';
 

package/src/networking/callTokenStanza.ts

@@ -57,6 +57,15 @@ const consumeKindHint = (
 
 const MAX_CALL_TOKEN_AGE_MS = 5000;
 
+// Dev-only trace of call signalling: which `<data type="call-*">` frames
+// arrive and why a call-token was dropped. A dropped token is silent
+// otherwise: the caller sits on "Calling..." and the callee never rings.
+const callTrace = (message: string, extra?: Record<string, unknown>) => {
+  if (typeof __DEV__ !== 'undefined' && __DEV__) {
+    console.log(`[calls] ${message}`, extra ? JSON.stringify(extra) : '');
+  }
+};
+
 // Hermes has no global `atob` and RN ships no Buffer polyfill by default,
 // so the JWT payload is decoded here by hand. Only used on the ~500-byte
 // middle segment of a call token, so a table-driven loop is plenty fast
@@ -215,7 +224,7 @@ const isFreshCallToken = (stanza: Element): boolean => {
 const resolveRoomByStanzaRoom = (roomAttr: string) => {
   const rooms = (store.getState().rooms.rooms || {}) as Record<
     string,
-    { jid?: string; name?: string; type?: string; usersCnt?: number }
+    { jid?: string; name?: string; title?: string; type?: string; usersCnt?: number }
   >;
   const roomEntries = Object.values(rooms);
 
@@ -338,6 +347,18 @@ export const onCallTokenMessage = (stanza: Element): boolean => {
   const data = findCallData(inner);
   if (data) {
     const dataType = String(data.attrs?.type || '').toLowerCase();
+    const traceStanzaId = inner.getChild?.('stanza-id')?.attrs?.id;
+    const traceTs = extractNumericTimestamp(traceStanzaId);
+    callTrace(`received ${dataType}`, {
+      from: stanza.attrs?.from,
+      type: stanza.attrs?.type,
+      wrapped: inner !== stanza,
+      room: data.attrs?.room,
+      callId: data.attrs?.callId || data.attrs?.callid,
+      hasToken: !!data.attrs?.token,
+      stanzaId: traceStanzaId,
+      ageMs: traceTs ? Date.now() - traceTs : null,
+    });
 
     if (dataType === 'call-state') {
       // Surface the raw call-state attrs so we can confirm what the server
@@ -404,9 +425,11 @@ export const onCallTokenMessage = (stanza: Element): boolean => {
       const videoCallsConfig =
         store.getState().chatSettingStore.config?.videoCalls;
       if (videoCallsConfig?.enabled !== true) {
+        callTrace('drop call-token: calls disabled');
         return true;
       }
       if (!isFreshCallToken(inner)) {
+        callTrace('drop call-token: not fresh');
         return true;
       }
 
@@ -423,6 +446,7 @@ export const onCallTokenMessage = (stanza: Element): boolean => {
       const kind: 'audio' | 'video' =
         hintedKind || (stanzaKind === 'audio' ? 'audio' : 'video');
       if (!token || !stanzaRoom) {
+        callTrace('drop call-token: no token or room');
         return true;
       }
 
@@ -436,9 +460,11 @@ export const onCallTokenMessage = (stanza: Element): boolean => {
         resolvedRoom.type !== 'private' &&
         !isOneToOneLikeRoom
       ) {
+        callTrace('drop call-token: room type', { type: resolvedRoom.type, usersCnt });
         return true;
       }
       if (resolvedRoom?.type === 'private' && !isPrivateAllowed) {
+        callTrace('drop call-token: private not allowed');
         return true;
       }
       if (
@@ -446,6 +472,7 @@ export const onCallTokenMessage = (stanza: Element): boolean => {
         !stanzaRoom.includes('@') &&
         !isOneToOneLikeRoom
       ) {
+        callTrace('drop call-token: room unknown', { stanzaRoom });
         return true;
       }
 
@@ -457,17 +484,25 @@ export const onCallTokenMessage = (stanza: Element): boolean => {
       // title (which for stale / deleted-peer chats reads "deleted").
       const jwtPayload = decodeJwtPayload(token);
       const callerName = extractCallerDisplayName(jwtPayload);
-      const resolvedTitle = String(resolvedRoom?.name || '').trim();
+      // `title` is the display name; on new-arch private rooms `name` is the
+      // bare room id ("${appId}_<a>-${appId}_<b>"), never something to show.
+      const resolvedTitle = String(
+        resolvedRoom?.title || resolvedRoom?.name || ''
+      ).trim();
       const titleLooksBad =
         !resolvedTitle ||
+        resolvedTitle === stanzaRoom ||
         ['deleted', 'deleted user', 'unknown', 'null'].includes(
           resolvedTitle.toLowerCase()
         );
+      // Never fall back to the room id: an empty name renders as
+      // "Unknown caller" on the ring screen.
       const incomingRoomName =
-        (callerName && titleLooksBad ? callerName : resolvedTitle) ||
-        callerName ||
-        stanzaRoom;
-      if (!roomJid) {return true;}
+        (titleLooksBad ? callerName : resolvedTitle) || callerName || '';
+      if (!roomJid) {
+        callTrace('drop call-token: no room jid', { stanzaRoom });
+        return true;
+      }
 
       const callId = extractCallId(data, jwtPayload);
       const selfXmpp =
@@ -481,6 +516,11 @@ export const onCallTokenMessage = (stanza: Element): boolean => {
           callState.roomName === incomingRoomName ||
           callState.roomBareName === stanzaRoom);
 
+      callTrace('accept call-token', {
+        outgoing: isOutgoingRoomMatch,
+        roomJid,
+        callId,
+      });
       if (isOutgoingRoomMatch) {
         store.dispatch(
           setOutgoingCallToken({ roomJid, token, callId: callId || null })

package/src/networking/xmppClient.ts

@@ -1,4 +1,4 @@
-import xmpp, { Client } from '@xmpp/client';
+import xmpp, { Client, xml } from '@xmpp/client';
 import { walletToUsername } from '../helpers/walletUsername';
 import { IMessage, xmppSettingsInterface } from '../types/types';
 
@@ -765,6 +765,15 @@ export class XmppClient {
       this.authRecoveryFailures = 0;
       this.authRejectionsSinceOnline = 0;
       this.reconnectAttempts = 0;
+      // Initial presence, as the web client sends on 'online'. Without it
+      // the session is never "available" to ejabberd, which then holds
+      // back every `type="chat"` message addressed to the bare JID: the
+      // call-invite and the call-token never arrived, so the phone neither
+      // rang nor joined its own outgoing call. Room traffic was unaffected
+      // (MUC joins and mucsub don't need it), which hid the problem.
+      Promise.resolve()
+        .then(() => this.client.send(xml('presence')))
+        .catch((err: unknown) => console.warn('initial presence failed', err));
       this.sendAllPresencesAndMarkReady();
       try {
         devPushLog(

package/src/version.ts

@@ -1,3 +1,3 @@
 // Generated by scripts/sync-version.mjs on `npm version`.
 // Edit package.json's version instead of this file.
-export const ETHORA_CHAT_COMPONENT_VERSION = '26.10.0';
+export const ETHORA_CHAT_COMPONENT_VERSION = '26.10.3';