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

Indexing errors show up as indexing not supported #15168

Closed
jryans opened this issue Sep 14, 2020 · 1 comment · Fixed by matrix-org/matrix-react-sdk#5787
Closed

Indexing errors show up as indexing not supported #15168

jryans opened this issue Sep 14, 2020 · 1 comment · Fixed by matrix-org/matrix-react-sdk#5787
Assignees
Labels
A-Indexing Indexing messages via Seshat P1 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@jryans
Copy link
Collaborator

jryans commented Sep 14, 2020

We should refine our error handling for event indexing. At the moment, if you have some error such as:

EventIndex: Error initializing the event index {"message":"Error opening the database: IndexError(IOError(IOError { path: None, err: Custom { kind: Other, error: \"Invalid MAC\" } }))"}

the Settings UI manifests this as if you are on web / a platform that does not support indexing at all:

At the very least, Settings UI should reflect that indexing is supported on your platform, but some error has occurred. Even better would be highlighting the error details as well.

@thomasfrobieter
Copy link

thomasfrobieter commented Feb 11, 2022

I still get:

Error opening the database: IndexError(IOError(IOError { path: None, err: Custom { kind: Other, error: "Invalid MAC" } }))

In Element Desktop (Win 10):
Version von Element: 1.10.1
Version von Olm: 3.2.8

UPDATE: Fixed by resetting the search-index with the button right under the error message in the Element > Security settings.

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 P1 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants