@droponair/sdk-js 0.32.0 → 0.32.1

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 0.32.1
+
+### Fixed
+
+- **A sender reported an error for every message it sent.** A message is echoed
+  back to every device of the sender, including the one that sent it, and what it
+  carries is encrypted for the recipients. The sending device tried to decrypt it,
+  which cannot work, and reported the failure each time. Such an echo is now
+  recognised and skipped. Nothing was lost before; the error was always wrong,
+  and an error that is always wrong hides the ones that are not.
+
+- **Reconnecting could stop for good.** A reconnect attempt that failed before a
+  socket existed, typically fetching a token with no network, emitted
+  `RECONNECT_FAILED` and scheduled nothing further, so the client stayed offline.
+  It now schedules the next attempt while reconnecting is still wanted.
+
 ## 0.32.0
 
 ### Added

package/dist/core/messaging-client.js

@@ -56,6 +56,12 @@ class MessagingClient {
             this.reconnectTimer = null;
             this.connectWebSocket().catch(() => {
                 this.emitEvent({ type: 'ERROR', reason: 'RECONNECT_FAILED' });
+                // An attempt that fails before a socket exists, typically the token exchange
+                // with no network, has no close event to schedule the next one. Without this
+                // the first such failure ended reconnecting for good.
+                if (this.shouldReconnect) {
+                    this.scheduleReconnect();
+                }
             });
         }, delay);
     }
@@ -2138,7 +2144,17 @@ class MessagingClient {
                 });
             }
             else {
-                // Legacy single-payload path
+                // Legacy single-payload path.
+                //
+                // Our own message, sent back so our other devices can show it. This client
+                // is the one that sent it and already has it, and what it carries is
+                // encrypted for the recipient, not for us. Trying anyway fails, and a
+                // failure that happens on every single message teaches people to ignore
+                // the ones that matter. A client with something to read here finds it in
+                // the per-device payloads above.
+                if (envelope.fromUserId === this.currentUserId) {
+                    return;
+                }
                 const sharedKey = await this.getPeerSharedKey(envelope.fromUserId);
                 plaintext = await this.cryptoService.decrypt(envelope.encryptedPayload, sharedKey, {
                     messageId: envelope.messageId,

package/dist/version.d.ts

@@ -7,7 +7,7 @@
  *   MINOR, additive feature (e.g. multi-device payloads, new call event type)
  *   PATCH, bug-fix / perf improvement with no wire or API change
  */
-export declare const SDK_VERSION = "0.32.0";
+export declare const SDK_VERSION = "0.32.1";
 /**
  * Binary encrypted-payload format version.
  * Included as the first byte of every encrypted payload so receivers can

package/dist/version.js

@@ -10,7 +10,7 @@ exports.PROTOCOL_VERSION = exports.PAYLOAD_FORMAT_VERSION = exports.SDK_VERSION
  *   MINOR, additive feature (e.g. multi-device payloads, new call event type)
  *   PATCH, bug-fix / perf improvement with no wire or API change
  */
-exports.SDK_VERSION = '0.32.0';
+exports.SDK_VERSION = '0.32.1';
 /**
  * Binary encrypted-payload format version.
  * Included as the first byte of every encrypted payload so receivers can

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@droponair/sdk-js",
-  "version": "0.32.0",
+  "version": "0.32.1",
   "description": "End-to-end encrypted messaging, voice and video calling SDK. The relay never sees your keys or message content.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",