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

Shift e2e processing into a webworker #2503

Open
ara4n opened this issue Oct 24, 2016 · 6 comments
Open

Shift e2e processing into a webworker #2503

ara4n opened this issue Oct 24, 2016 · 6 comments
Labels
A-E2EE A-Performance P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Enhancement

Comments

@ara4n
Copy link
Member

ara4n commented Oct 24, 2016

We shouldn't be locking up the UI when encrypting/decrypting E2E; shouldn't this be in a bg worker?

@richvdh richvdh added the A-E2EE label Oct 26, 2016
@richvdh
Copy link
Member

richvdh commented Oct 26, 2016

Yes it should, but that will mean asynchronous decryption of events, which will probably mean hooks up to the react layer.

@richvdh richvdh added feature A-Performance P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround S-Minor Impairs non-critical functionality or suitable workarounds exist and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Oct 31, 2016
@ara4n
Copy link
Member Author

ara4n commented Jan 14, 2018

now the whole API is async thanks to indexeddb, does this become easier? Or have we even got it for free?

@marcusmueller
Copy link

Since this has become the blocker issue for #26231 (which is a paying-customer facing usability regression), I'd like to understand what the status of this issue is, and whether we are "close" in temporal terms.

@richvdh
Copy link
Member

richvdh commented Oct 2, 2023

@marcusmueller I'm afraid it's unlikely to be a priority within the next few weeks.

@marcusmueller
Copy link

Now, I understand that weeks are short. I feel like I'm touting my own horn here a bit hard, but if this change does not end up being a priority at some point in the medium future, that'd be a bit sad:

We have a significant UX regression, which was brought by the necessity of being safe around rust-sdk-crypto not being concurrency-safe, and the plan would be neither to fundamentally change rust-sdk nor element-web to actually work around this fact, but live with the regression that users can no longer use the same session in multiple tabs.

@lunera-dev
Copy link

It's 7+ months later and this is still an issue. In my use case, I am NOT using E2E because encryption in my org is against policy. Is there a way I can have multiple tabs (rooms) open at the same time since E2E is disabled? #26231 links here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE A-Performance P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Enhancement
Projects
None yet
Development

No branches or pull requests

5 participants