-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
chore: Force logout live clients on E2EE keys reset changes #32291
Conversation
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #32291 +/- ##
===========================================
+ Coverage 55.36% 55.42% +0.05%
===========================================
Files 2330 2329 -1
Lines 51392 51361 -31
Branches 10516 10510 -6
===========================================
+ Hits 28453 28466 +13
+ Misses 20416 20378 -38
+ Partials 2523 2517 -6
Flags with carried forward coverage won't be shown. Click here to find out more. |
Looks like this PR is ready to merge! 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀
…engagementMDM * 'develop' of github.com:RocketChat/Rocket.Chat: chore: including x.com hostname to Twitter oembed (#32267) feat: node-gcm deprecation (#32208) chore: Deprecate mute/unmute meteor methods and convert to endpoint (#31811) fix: Wrong Business hours validations between different weeks (#32287) chore: Force logout live clients on E2EE keys reset changes (#32291) ci: skip update version message for TEST_MODE=true (#32293)
Proposed changes (including videos or screenshots)
Only counting on client to logout is a weak strategy, so now we are closing the connections properly on microservices
Issue(s)
Introduced here: #31958
E2EE-10
Steps to test or reproduce
Further comments