This differential is created in response to the following linear issue: https://linear.app/comm/issue/ENG-4451/improve-logging-for-olm-session-creation-failure. Olm session creation failure is a seriious concern and current error handling does
not provide meaningful information for debugging purposes.
Details
Details
Delete olm sessions table from the database. Try to log in from mobile app. Ensure that meaningful logs are printed to the console.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
keyserver/src/creators/olm-session-creator.js | ||
---|---|---|
27 ↗ | (On Diff #29010) | We probably shouldn't log cookieID |
keyserver/src/creators/olm-session-creator.js | ||
---|---|---|
27 ↗ | (On Diff #29010) | Knowing the cookieID for which the failure occurred might be helpful for debugging purposes. Does it create security issues? |
keyserver/src/creators/olm-session-creator.js | ||
---|---|---|
27 ↗ | (On Diff #29010) |
|
keyserver/src/creators/olm-session-creator.js | ||
---|---|---|
27 ↗ | (On Diff #29057) | Missing a space at the end |