Paths

Table of Contentst

Differential D11719

[web/lib] add sequential decrypt to `olmAPI`
ClosedPublic
Actions

Authored by kamil on Apr 22 2024, 3:42 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, May 18, 11:10 PM

	Unknown Object (File)
	Sat, May 11, 6:14 PM

	Unknown Object (File)
	Sat, May 4, 11:56 AM

	Unknown Object (File)
	Fri, May 3, 11:12 PM

	Unknown Object (File)
	Sun, Apr 28, 5:36 PM

	Unknown Object (File)
	Apr 26 2024, 4:03 PM

	Unknown Object (File)
	Apr 23 2024, 1:50 AM

	Unknown Object (File)
	Apr 23 2024, 1:49 AM

View All 9 Files

Subscribers

Details

Reviewers

tomek
marcin

Commits

rCOMMa14e4c1d5e9a: [web/lib] add sequential decrypt to `olmAPI`

Summary

Binding to olm method to sequential decrypt. Implementing different because this method will also be responsible for persisting message in SQLite (added later in the stack).

Depends on D11718

Test Plan

Created session between two web clients:

Sending couple of encrypted messsages
Sending messages with wrong order -> OLM_MESSAGE_OUT_OF_ORDER
Sending the same message multiple times -> OLM_ALREADY_DECRYPTED_OR_KEYS_SKIPPED

Diff Detail

Repository

Branch

publish-receiver

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Apr 22 2024, 3:42 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptApr 22 2024, 3:42 PM

kamil added a child revision: D11720: [lib] update P2P message handler to use sequential decrypt.Apr 22 2024, 3:53 PM

Harbormaster completed remote builds in B28376: Diff 39370.Apr 22 2024, 4:01 PM

kamil added inline comments.Apr 22 2024, 5:21 PM

web/shared-worker/worker/worker-crypto.js
470	mistake while rebasing

kamil added inline comments.Apr 23 2024, 5:27 AM

web/shared-worker/worker/worker-crypto.js
459	removed in D11728

kamil published this revision for review.Apr 23 2024, 6:38 AM

marcin accepted this revision.Apr 24 2024, 3:11 AM

This revision is now accepted and ready to land.Apr 24 2024, 3:11 AM

decrypt -> decrypt_sequential

Harbormaster completed remote builds in B28555: Diff 39612.Mon, Apr 29, 1:04 AM

Closed by commit rCOMMa14e4c1d5e9a: [web/lib] add sequential decrypt to `olmAPI` (authored by kamil). · Explain WhyMon, Apr 29, 1:11 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMa14e4c1d5e9a: [web/lib] add sequential decrypt to `olmAPI`.

Revision Contents
Changeset List

Path

Size

lib/

types/

crypto-types.js

5 lines

utils/

__mocks__/

1 line

native/

crypto/

1 line

web/

crypto/

1 line

shared-worker/

worker/

worker-crypto.js

24 lines

Diff 39370

lib/types/crypto-types.js

Loading...

lib/utils/mocks/config.js

Loading...

native/crypto/olm-api.js

Loading...

web/crypto/olm-api.js

Loading...

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

Loading...