Page MenuHomePhabricator
Differential D13898

[web] Implement getInboundP2PMessagesByID query on worker
ClosedPublic
Actions

Authored by angelika on Nov 8 2024, 7:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 13, 11:16 PM
Unknown Object (File)
Fri, Dec 13, 11:16 PM
Unknown Object (File)
Fri, Dec 13, 11:16 PM
Unknown Object (File)
Fri, Dec 13, 11:16 PM
Unknown Object (File)
Fri, Dec 13, 11:15 PM
Unknown Object (File)
Fri, Dec 13, 11:09 PM
Unknown Object (File)
Mon, Dec 9, 9:29 AM
Unknown Object (File)
Fri, Dec 6, 1:08 AM
View All 49 Files
Subscribers
ashoat

Details

Reviewers
tomek
kamil
Commits
rCOMM56ed820efec3: [web] Implement getInboundP2PMessagesByID query on worker
Summary

https://linear.app/comm/issue/ENG-9713/mitigate-risk-of-olm-already-decrypted-or-keys-skipped

Depends on D13897

Test Plan

Test plan in the last diff in stack.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

angelika created this revision.Nov 8 2024, 7:52 AM
Herald added a subscriber: ashoat. · View Herald TranscriptNov 8 2024, 7:52 AM
angelika added a child revision: D13899: [native] Implement getInboundP2PMessagesByID query in JSI.Nov 8 2024, 7:55 AM
Harbormaster completed remote builds in B32593: Diff 45722.Nov 8 2024, 8:16 AM
angelika requested review of this revision.Nov 8 2024, 8:16 AM
kamil accepted this revision.Nov 12 2024, 2:47 AM
kamil added inline comments.
web/types/worker-types.js
276 ↗(On Diff #45722)

note for myself (and other reviewers) - there is no need to add response type because we can reuse this type, that's why Flow is not complaining

This revision is now accepted and ready to land.Nov 12 2024, 2:47 AM
angelika updated this revision to Diff 45736.Nov 12 2024, 4:50 AM

Rebase

Harbormaster failed remote builds in B32602: Diff 45736!Nov 12 2024, 5:18 AM
angelika updated this revision to Diff 45741.Nov 12 2024, 5:44 AM

Rebase

Harbormaster completed remote builds in B32607: Diff 45741.Nov 12 2024, 6:18 AM
Closed by commit rCOMM56ed820efec3: [web] Implement getInboundP2PMessagesByID query on worker (authored by angelika). · Explain WhyNov 13 2024, 4:17 AM
This revision was automatically updated to reflect the committed changes.
angelika added a commit: rCOMM56ed820efec3: [web] Implement getInboundP2PMessagesByID query on worker.

Revision Contents
Changeset List

PathSize
web/
shared-worker/
worker/
9 lines
types/
9 lines

Diff 45741

View Options

web/shared-worker/worker/shared-worker.js

Loading...
View Options

web/types/worker-types.js

Loading...