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

Search doesn't seem to index encrypted rooms #28683

Closed
krokodilerian opened this issue Dec 8, 2024 · 2 comments
Closed

Search doesn't seem to index encrypted rooms #28683

krokodilerian opened this issue Dec 8, 2024 · 2 comments
Labels
A-Indexing Indexing messages via Seshat O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@krokodilerian
Copy link

krokodilerian commented Dec 8, 2024

Steps to reproduce

This is a new, fresh installation of element-desktop, I've been using the web version until now (but needed search).

I start element-desktop in a terminal, and I see

Starting auto update with base URL: https://packages.element.io/desktop/update/
Auto update not supported on this platform
Fetching translation json for locale: en_EN
Changing application language to en
Fetching translation json for locale: en
Resetting the UI components after locale change
Resetting the UI components after locale change
Changing application language to en
Fetching translation json for locale: en
Resetting the UI components after locale change
Error getting the event index passphrase out of the secret store TypeError: keytar.setPassword is not a function
    at getOrCreatePassphrase (file:///opt/Element/resources/app.asar/lib/seshat.js:42:30)
    at async IpcMainImpl.<anonymous> (file:///opt/Element/resources/app.asar/lib/seshat.js:90:36)
(node:3543159) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 serverSupportedVersions listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(Use `element-desktop --trace-warnings ...` to show where the warning was created)
(node:3543159) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 userAccessToken listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(node:3543159) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 homeserverUrl listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(node:3543159) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 userAccessToken listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit

I then go to Setting->Security and privacy->Search, and check the state of the message sync. It's done after a while, but with less messages than I expect.

Then, I try a search in a private chat, for words I know exist there, and nothing is found. I can see messages being found only in one of the non-encrypted channels I'm in, if I ask for Element to search in all channels.

Outcome

What did you expect?

To see messages from private channels being found with search

What happened instead?

No such messages were found

Operating system

Debian stable

Application version

Element version: 1.11.87 Crypto version: Rust SDK 0.7.2 (517d99b), Vodozemac 0.7.0

How did you install the app?

https://element.io/download

Homeserver

fosdem.ems.host

Will you send logs?

Yes

@dosubot dosubot bot added A-Indexing Indexing messages via Seshat O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Dec 8, 2024
@Sazu-bit
Copy link

Sazu-bit commented Dec 8, 2024

I get this with opening an attachment.

No update_base_url is defined: auto update is disabled
Fetching translation json for locale: en_EN
Changing application language to en
Fetching translation json for locale: en
Resetting the UI components after locale change
Resetting the UI components after locale change
Changing application language to en
Fetching translation json for locale: en
Resetting the UI components after locale change
Error getting the event index passphrase out of the secret store TypeError: keytar.setPassword is not a function
    at getOrCreatePassphrase (file:///usr/lib/element-desktop/resources/app/lib/seshat.js:42:30)
    at async IpcMainImpl.<anonymous> (file:///usr/lib/element-desktop/resources/app/lib/seshat.js:90:36)
(node:25831) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 serverSupportedVersions listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(Use `electron --trace-warnings ...` to show where the warning was created)
(node:25831) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 userAccessToken listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(node:25831) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 serverSupportedVersions listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(node:25831) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 userAccessToken listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(node:25831) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 serverSupportedVersions listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
(node:25831) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 userAccessToken listeners added to [IpcMainImpl]. MaxListeners is 10. Use emitter.setMaxListeners() to increase limit
Crashing due to FD ownership violation:
#0 0x60d634a595d4 <unknown>
element-hq/element-web#1 0x60d634a5084e <unknown>
element-hq/element-web#2 0x60d634a50802 close
element-hq/element-web#3 0x7a64b6af2584 pa_close
element-hq/element-web#4 0x60d634a7f7cc <unknown>
element-hq/element-web#5 0x60d634a1909a <unknown>
element-hq/element-web#6 0x60d6349e5e9a <unknown>
element-hq/element-web#7 0x60d63383cb9d <unknown>
element-hq/element-web#8 0x60d63383e9f3 <unknown>
element-hq/element-web#9 0x60d63383a04c <unknown>
element-hq/element-web#10 0x60d6312401a7 <unknown>
element-hq/element-web#11 0x60d631241b29 <unknown>
element-hq/element-web#12 0x60d631241918 <unknown>
element-hq/element-web#13 0x60d63123ecb1 <unknown>
element-hq/element-web#14 0x60d63123f1aa <unknown>
element-hq/element-web#15 0x60d630ee323e <unknown>
element-hq/element-web#16 0x7a64b9335d6e <unknown>

Looks like someone coded the keytar.setPassword call incorrectly.

@t3chguy
Copy link
Member

t3chguy commented Dec 9, 2024

Looks like someone coded the keytar.setPassword call incorrectly.

image

It hasn't changed in over 2 years

Duplicate of element-hq/element-desktop#2028

@t3chguy t3chguy closed this as not planned Won't fix, can't repro, duplicate, stale Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Indexing Indexing messages via Seshat O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests

3 participants