-
Notifications
You must be signed in to change notification settings - Fork 359
Enable trace logging for the Olm Account #2357
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
Conversation
This should help us to track down UTD sources, it ensures that we log the state of a Olm session when we try to decrypt an `m.olm.*` to-device message.
|
Thank you for your contribution! Here are a few things to check in the PR to ensure it's reviewed as quickly as possible:
|
|
I'll create a new SDK version then, if those changes are needed. |
Oh, they aren't strictly needed, but they would be helpful since we otherwise still miss some nice information. |
|
The Rust SDK version that contains the changes for this PR is in #2359 |
Thanks for taking care of the SDK bump. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2357 +/- ##
========================================
Coverage 71.28% 71.29%
========================================
Files 1352 1353 +1
Lines 31942 31962 +20
Branches 6339 6341 +2
========================================
+ Hits 22769 22786 +17
Misses 5863 5863
- Partials 3310 3313 +3 ☔ View full report in Codecov by Sentry. |
jmartinesp
left a comment
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.
It works fine, thanks!
This should help us to track down UTD sources, it ensures that we log the state of a Olm session when we try to decrypt an
m.olm.*to-device message.This should be applied after we update to matrix-org/matrix-rust-sdk#3100.
Matching EX-iOS PR: element-hq/element-x-ios#2427