Browse Source

Fixes #2500

This wrongly stored value wasn't inlcuded in the published the bundle
because the libsignal store was used, which had the right value for the public key.

Instead, this value was used locally by being passed to the libsignal
session builder to verify signed prekey.
JC Brand 4 years ago
parent
commit
c966c49304
2 changed files with 6 additions and 2 deletions
  1. 5 1
      CHANGES.md
  2. 1 1
      src/plugins/omemo/store.js

+ 5 - 1
CHANGES.md

@@ -42,7 +42,11 @@ Removed events:
 The `chatBoxClosed`, `chatBoxMaximized` and `chatBoxMinimized` events now have the `model` as
 payload and not the `view`.
 
-## 7.0.5 (Unreleased)
+## 7.0.6 (unreleased)
+
+- #2500: Wrong assignment in OMEMO code
+
+## 7.0.5 (2021-03-16)
 
 - #2377: The @converse/headless NPM package is missing the dist directory, causing import errors
 - #2396: @converse/headless wrongly depends on `CustomElement` from the view layer

+ 1 - 1
src/plugins/omemo/store.js

@@ -237,7 +237,7 @@ const OMEMOStore = Model.extend({
         _converse.omemo_store.storeSignedPreKey(signed_prekey);
         bundle['signed_prekey'] = {
             'id': signed_prekey.keyId,
-            'public_key': u.arrayBufferToBase64(signed_prekey.keyPair.privKey),
+            'public_key': u.arrayBufferToBase64(signed_prekey.keyPair.pubKey),
             'signature': u.arrayBufferToBase64(signed_prekey.signature)
         };
         const keys = await Promise.all(