Page MenuHomePhabricator
Differential D13430

[native] Improve logging for thick threads notifs
ClosedPublic
Actions

Authored by ashoat on Sep 23 2024, 10:14 PM.
Tags
None
Referenced Files
F3515845: D13430.id44453.diff
Sun, Dec 22, 10:33 AM
Unknown Object (File)
Thu, Dec 5, 7:50 PM
Unknown Object (File)
Thu, Dec 5, 7:44 PM
Unknown Object (File)
Thu, Dec 5, 7:44 PM
Unknown Object (File)
Thu, Dec 5, 6:19 PM
Unknown Object (File)
Nov 18 2024, 5:45 PM
Unknown Object (File)
Nov 8 2024, 7:02 PM
Unknown Object (File)
Nov 8 2024, 7:02 PM
View All 32 Files
Subscribers
tomek

Details

Reviewers
marcin
Commits
rCOMM7eea1cd5e976: [native] Improve logging for thick threads notifs
Summary

I found these logging improvements helpful while investigating ENG-9233. I'll annotate each inline.

Test Plan

Confirmed logging improvements

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat created this revision.Sep 23 2024, 10:14 PM
Herald added a subscriber: tomek. · View Herald TranscriptSep 23 2024, 10:14 PM
ashoat added a child revision: D13431: [native] Avoid Promise.allSettled.Sep 23 2024, 10:16 PM
ashoat added inline comments.Sep 23 2024, 10:22 PM
lib/push/crypto.js
192

Before:

Notification encryption failed: [object Object]

After:

Notification encryption failed {"message": "Session with deviceID: tjiSvK/6Z1aebjrLb55mpzWoIiN4Lr3yndNvNYw7ySo not initialized."}
lib/push/send-utils.js
1001

Before:

[TypeError: undefined is not a function]

After:

Olm session creation failed [TypeError: undefined is not a function]
Harbormaster completed remote builds in B31787: Diff 44453.Sep 23 2024, 10:31 PM
ashoat requested review of this revision.Sep 23 2024, 10:31 PM
marcin accepted this revision.Sep 24 2024, 9:16 AM
This revision is now accepted and ready to land.Sep 24 2024, 9:16 AM
Closed by commit rCOMM7eea1cd5e976: [native] Improve logging for thick threads notifs (authored by ashoat). · Explain WhySep 24 2024, 3:04 PM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM7eea1cd5e976: [native] Improve logging for thick threads notifs.

Revision Contents
Changeset List

PathSize
lib/
push/
8 lines
2 lines

Diff 44453

View Options

lib/push/crypto.js

Loading...
View Options

lib/push/send-utils.js

Loading...