1634436 - Add an encrypted IPC serializable stream implementation

Status: RESOLVED FIXED

(Core :: Storage: IndexedDB, task, P2)

Comment 1

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1634436 - Add NSSCipherStrategy. r=#dom-workers-and-storage

Comment 2

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1634436 - Add support for seeking (nsISeekableStream). r=#dom-workers-and-storage

Comment 3

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1634436 - Add encrypted stream implementation. r=#dom-workers-and-storage

Comment 4

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1634436 - Make DecryptingInputStream implement nsICloneableInputStream. r=#dom-workers-and-storage

Comment 5

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1634436 - Make DecryptingInputStream implement nsIIPCSerializableInputStream. r=#dom-workers-and-storage

Comment 6

[Phabricator Automation]

Comment on attachment 9144720 [details]
Bug 1634436 - Add NSSCipherStrategy. r=#dom-workers-and-storage

Revision D73290 was moved to bug 1641178. Setting attachment 9144720 [details] to obsolete.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2bdc7d7249f5
Add encrypted stream implementation. r=dom-workers-and-storage-reviewers,janv
https://hg.mozilla.org/integration/autoland/rev/dc8cff56948f
Add support for seeking (nsISeekableStream). r=dom-workers-and-storage-reviewers,janv

Comment 8

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5c8df4a1f5c4
Backed out 2 changesets for causing bustages in TestEncryptedStream.cpp

Comment 9

[Alexandru Michis [:malexandru]]

Backed out 2 changesets for causing bustages in TestEncryptedStream.cpp

Backout link: https://hg.mozilla.org/integration/autoland/rev/5c8df4a1f5c413abdaf45cbc466ffc428f4e965b

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&searchStr=linux%2Cx64%2Cdebug%2Cbuild-linux64-base-toolchains%2Fdebug%2C%28bb%29&tochange=ab74bac09a68daab7a7e94ad5ffd6b476a704777&fromchange=fab7c4f54054ceb06504c7ddac380858e2521fc4&selectedTaskRun=J-B6j1HGTxeuQYGBLhuFVw-0

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=305843060&repo=autoland&lineNumber=24639

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e9108c87470a
Add encrypted stream implementation. r=dom-workers-and-storage-reviewers,janv
https://hg.mozilla.org/integration/autoland/rev/14681da50635
Add support for seeking (nsISeekableStream). r=dom-workers-and-storage-reviewers,janv

Comment 11

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/e9108c87470a
https://hg.mozilla.org/mozilla-central/rev/14681da50635

Comment 12

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1634436 - Fix DecryptingInputStream::Available and add tests. r=#dom-workers-and-storage,janv

Comment 13

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1634436 - Make cipher-strategy stateful and keep mode, key and IV as state. r=#dom-workers-and-storage

Depends on D75913

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b952ecd5b548
Fix DecryptingInputStream::Available and add tests. r=janv,dom-workers-and-storage-reviewers
https://hg.mozilla.org/integration/autoland/rev/cf8febbd632f
Make DecryptingInputStream implement nsICloneableInputStream. r=dom-workers-and-storage-reviewers,janv
https://hg.mozilla.org/integration/autoland/rev/527feee80bc9
Make DecryptingInputStream implement nsIIPCSerializableInputStream. r=baku

Comment 15

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/b952ecd5b548
https://hg.mozilla.org/mozilla-central/rev/cf8febbd632f
https://hg.mozilla.org/mozilla-central/rev/527feee80bc9

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/199f5f63f63c
Make cipher-strategy stateful and keep mode, key and IV as state. r=dom-workers-and-storage-reviewers,janv

Comment 17

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/199f5f63f63c