[web-db] re-create encryption key from JWK on Safari
ClosedPublic
Actions

Authored by kamil on Jun 6 2023, 10:19 AM.

Details

Reviewers

michal
tomek

Commits

rCOMM79937bdb748f: [web-db] re-create encryption key from JWK on Safari

Summary

Using key created in main thread.
(context)

Depends on D8121

Test Plan

Check if key is properly interpreted

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Jun 6 2023, 10:19 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJun 6 2023, 10:19 AM

kamil added a child revision: D8124: [web-db] avoid database init on app start on Safari.Jun 6 2023, 10:20 AM

Harbormaster completed remote builds in B20060: Diff 27490.Jun 6 2023, 10:35 AM

kamil published this revision for review.Jun 7 2023, 1:50 AM

tomek accepted this revision.Jun 13 2023, 4:29 AM

tomek added inline comments.

web/database/worker/db-worker.js
65 ↗	(On Diff #27490)	We don't need to explicitly state Safari here - this function doesn't care about the platform

This revision is now accepted and ready to land.Jun 13 2023, 4:29 AM

rename param

web/database/worker/db-worker.js
65 ↗	(On Diff #27490)	`encryptionKey` is already declared (see line 57), I'll update this to `encryptionKeyJWK`

Harbormaster completed remote builds in B20252: Diff 27752.Jun 14 2023, 9:36 AM

Closed by commit rCOMM79937bdb748f: [web-db] re-create encryption key from JWK on Safari (authored by kamil). · Explain WhyJun 14 2023, 10:14 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM79937bdb748f: [web-db] re-create encryption key from JWK on Safari.

Revision Contents
Changeset List

Path

Size

web/

database/

worker/

db-worker.js

25 lines

Diff 27764

View Options

web/database/worker/db-worker.js

[web-db] re-create encryption key from JWK on SafariClosedPublicActions