Page MenuHomePhabricator
Differential D6994

[web-db] implement types for `SubtleCrypto` module
ClosedPublic
Actions

Authored by kamil on Mar 7 2023, 9:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Aug 26, 4:49 PM
Unknown Object (File)
Sat, Aug 26, 5:15 AM
Unknown Object (File)
Fri, Aug 25, 5:24 PM
Unknown Object (File)
Fri, Aug 25, 5:01 AM
Unknown Object (File)
Thu, Aug 24, 5:28 PM
Unknown Object (File)
Thu, Aug 24, 3:57 PM
Unknown Object (File)
Jun 23 2023, 1:06 AM
Unknown Object (File)
Jun 23 2023, 1:06 AM
View All 23 Files
Subscribers
ashoat
atul
bartek

Details

Reviewers
michal
tomek
bartek
ashoat
Commits
rCOMM498483793f2e: [web-db] implement types for `SubtleCrypto` module
Summary

There are no build in flow types for SubtleCrypto or any implementation we can use.

Adding types based on API docs and typescript.

Test Plan

N/A

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Mar 7 2023, 9:31 AM
kamil held this revision as a draft.
Herald added subscribers: atul, ashoat. · View Herald TranscriptMar 7 2023, 9:31 AM
kamil added a child revision: D6995: [web-db] implement crypto functions.Mar 7 2023, 9:35 AM
Harbormaster completed remote builds in B17170: Diff 23512.Mar 7 2023, 9:47 AM
kamil published this revision for review.Mar 7 2023, 10:08 AM
ashoat added inline comments.Mar 7 2023, 4:32 PM
web/database/utils/crypto-types.js
138–263 ↗(On Diff #23512)

Why aren't these ones read-only?

268–269 ↗(On Diff #23512)

Why aren't these ones read-only?

michal added inline comments.Mar 8 2023, 4:49 AM
web/database/utils/crypto-types.js
4 ↗(On Diff #23512)

Can this be read-only?

bartek mentioned this in D7023: [web] Add Flow types for WebCrypto / SubtleCrypto.Mar 9 2023, 11:39 AM
kamil updated this revision to Diff 23601.Mar 10 2023, 7:04 AM

improve types and move to flow-typed

kamil added reviewers: bartek, ashoat.Mar 10 2023, 7:11 AM
kamil added a subscriber: bartek.

Previous types were too general (string instead of current algorithm names). I've updated this to be more precise (like D7023) but with specific types for the entire module - not only AES algorithms.

Checked if it works with D7024 (cc. @bartek).

Harbormaster completed remote builds in B17231: Diff 23601.Mar 10 2023, 7:20 AM
ashoat accepted this revision.Mar 10 2023, 10:51 AM
bartek accepted this revision.Mar 12 2023, 11:48 PM

LGTM, Works with my changes. Thank you for taking care of this!

This revision is now accepted and ready to land.Mar 12 2023, 11:48 PM
bartek added a child revision: D7024: [web] Implement AES encryption functions.Mar 12 2023, 11:49 PM
Closed by commit rCOMM498483793f2e: [web-db] implement types for `SubtleCrypto` module (authored by kamil). · Explain WhyMar 13 2023, 1:13 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM498483793f2e: [web-db] implement types for `SubtleCrypto` module.
kamil mentioned this in D8470: [web] rename WebCrypto to match convention.Jul 11 2023, 4:14 AM
kamil mentioned this in rCOMMfa7c84c9347a: [web] rename WebCrypto to match convention.Jul 12 2023, 6:07 AM

Revision Contents
Changeset List

PathSize
web/
flow-typed/
285 lines
utils/
5 lines

Diff 23664

View Options

web/flow-typed/WebCrypto.js

Loading...
View Options

web/utils/text-utils.js

Loading...