Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Element-R: log spam when viewing Unable-to-decrypt event #26272

Closed
richvdh opened this issue Oct 2, 2023 · 1 comment · Fixed by matrix-org/matrix-js-sdk#3824
Closed

Element-R: log spam when viewing Unable-to-decrypt event #26272

richvdh opened this issue Oct 2, 2023 · 1 comment · Fixed by matrix-org/matrix-js-sdk#3824
Assignees
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect Team: Crypto

Comments

@richvdh
Copy link
Member

richvdh commented Oct 2, 2023

Whenever you open a room with a UISI event, this is spammed out to the logs:

Error getting encryption info on event Error: Can't find the room key to decrypt the event, withheld code: None
    __wbindgen_error_new matrix_sdk_crypto_wasm.js:6792
    __wbg_adapter_59 matrix_sdk_crypto_wasm.js:274
    real matrix_sdk_crypto_wasm.js:216
    promise callback*1796/</module.exports.__wbg_then_11f7a54d67b4bfad matrix_sdk_crypto_wasm.js:7573
    __wbg_adapter_56 matrix_sdk_crypto_wasm.js:270
    real matrix_sdk_crypto_wasm.js:252
    __wbg_setonsuccess_5f71593bc51653a3 matrix_sdk_crypto_wasm.js:7192
    __wbg_adapter_59 matrix_sdk_crypto_wasm.js:274
    real matrix_sdk_crypto_wasm.js:216
    promise callback*1796/</module.exports.__wbg_then_11f7a54d67b4bfad matrix_sdk_crypto_wasm.js:7573
    __wbg_adapter_56 matrix_sdk_crypto_wasm.js:270
    real matrix_sdk_crypto_wasm.js:252
    __wbg_setonsuccess_5f71593bc51653a3 matrix_sdk_crypto_wasm.js:7192
@richvdh richvdh added T-Defect S-Tolerable Low/no impact on users O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience Team: Crypto A-Element-R Issues affecting the port of Element's crypto layer to Rust labels Oct 2, 2023
@richvdh
Copy link
Member Author

richvdh commented Oct 23, 2023

There is a similar spam when viewing a message that is still being sent:

rageshake.ts:74 Error getting encryption info on event ~!UcgyhoigetVICUfvRw:matrix.org:m1698055312450.2 in room !UcgyhoigetVICUfvRw:matrix.org Error: leading sigil is incorrect or missing at line 1 column 62
    at push.1407.module.exports.__wbindgen_error_new (matrix_sdk_crypto_wasm.js:6834:1)
    at 011935b2:0x3c1df7
    at 011935b2:0x3d8214
    at 011935b2:0xb096c
    at 011935b2:0x24716f
    at 011935b2:0x294950
    at 011935b2:0x3f080a
    at __wbg_adapter_59 (matrix_sdk_crypto_wasm.js:274:1)
    at real (matrix_sdk_crypto_wasm.js:216:1)

@richvdh richvdh self-assigned this Oct 23, 2023
richvdh added a commit to matrix-org/matrix-js-sdk that referenced this issue Oct 23, 2023
richvdh added a commit to matrix-org/matrix-js-sdk that referenced this issue Oct 23, 2023
github-merge-queue bot pushed a commit to matrix-org/matrix-js-sdk that referenced this issue Oct 23, 2023
* Element-R: silence log errors when viewing a pending event

Fixes the second half of element-hq/element-web#26272

* Update spec/integ/crypto/crypto.spec.ts
github-merge-queue bot pushed a commit to matrix-org/matrix-js-sdk that referenced this issue Oct 25, 2023
* Element-R: silence log errors when viewing a pending event

Fixes the second half of element-hq/element-web#26272

* Update spec/integ/crypto/crypto.spec.ts
github-merge-queue bot pushed a commit to matrix-org/matrix-js-sdk that referenced this issue Oct 25, 2023
* Element-R: silence log errors when viewing a pending event

Fixes the second half of element-hq/element-web#26272

* Update spec/integ/crypto/crypto.spec.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect Team: Crypto
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant