Page MenuHomePhabricator
Differential D7061

[lib] Introduce combined padding interface
ClosedPublic
Actions

Authored by bartek on Mar 13 2023, 7:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 4, 2:12 AM
Unknown Object (File)
Wed, Dec 4, 1:32 AM
Unknown Object (File)
Wed, Dec 4, 1:28 AM
Unknown Object (File)
Tue, Dec 3, 12:06 PM
Unknown Object (File)
Nov 10 2024, 7:02 PM
Unknown Object (File)
Nov 9 2024, 10:18 PM
Unknown Object (File)
Nov 8 2024, 9:23 PM
Unknown Object (File)
Nov 8 2024, 9:23 PM
View All 84 Files
Subscribers
anunay
ashoat
tomek

Details

Reviewers
atul
marcin
tomek
Commits
rCOMMa37f8d4289a4: [lib] Introduce combined padding interface
Summary

Part of ENG-3099

This diff is combining two previous diffs and finally allows to pad the data to multiples of 10KB (forty 250-byte blocks)

The whole idea is described in this Notion doc.

Depends on D7060

Test Plan

Added unit tests. Also tested using my CLI tool and hex editor / viewer (e.g. xxd tool). The CLI tool is also available in the Notion doc above

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek created this revision.Mar 13 2023, 7:46 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptMar 13 2023, 7:46 AM
bartek added inline comments.Mar 13 2023, 7:50 AM
lib/utils/pkcs7-padding.js
172–178 ↗(On Diff #23693)

This might be considered as an anti-pattern but I have no better idea (besides simply exporting them all)

Harbormaster completed remote builds in B17294: Diff 23693.Mar 13 2023, 8:02 AM
bartek published this revision for review.Mar 13 2023, 8:11 AM
bartek removed a reviewer: anunay.
bartek added a subscriber: anunay.
atul added a reviewer: tomek.Mar 16 2023, 1:37 PM

Specifically adding @tomek as reviewer since he seemed to have some thoughts during the Encryption Sync (that might've been discussed offline?)

atul accepted this revision.Mar 20 2023, 7:07 AM
This revision is now accepted and ready to land.Mar 20 2023, 7:07 AM
bartek updated this revision to Diff 23948.Mar 21 2023, 9:28 AM

Rebase, fix exports

Harbormaster completed remote builds in B17483: Diff 23948.Mar 21 2023, 9:44 AM
Closed by commit rCOMMa37f8d4289a4: [lib] Introduce combined padding interface (authored by bartek). · Explain WhyMar 21 2023, 10:21 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMa37f8d4289a4: [lib] Introduce combined padding interface.

Revision Contents
Changeset List

PathSize
lib/
utils/
60 lines
70 lines

Diff 23948

View Options

lib/utils/pkcs7-padding.js

Loading...
View Options

lib/utils/pkcs7-padding.test.js

Loading...