Skip to content

Conversation

@trop
Copy link
Contributor

@trop trop bot commented Aug 5, 2021

Backport of #30020

See that PR for details.

Notes: add safeStorage string encryption API

@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Aug 5, 2021
@trop trop bot added 15-x-y backport This is a backport PR backport/requested 🗳 semver/minor backwards-compatible functionality labels Aug 5, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Aug 5, 2021
Copy link
Member

@VerteDinde VerteDinde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Making sure this gets API WG approval before merging API WG isn't need for backports, we're good to go! 👍

Copy link
Member

@MarshallOfSound MarshallOfSound left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved @electron/wg-releases

* feat: add SafeStorage api; first commit

* chore: rename files to fit semantically

* chore: add linkedBindings

* chore: fix function signatures

* chore: refactor eisCookieEncryptionEnabled() fuse

* chore: create test file

* chore: add tests and documentation

* chore: add copyright and lint

* chore: add additional tests

* chore: fix constructor

* chore: commit for pair programming

* wip: commit for keeley pairing

* chore: docs change and code cleanup

* chore: add linux import

* chore: add description to documentation

* chore: fixing tests

* chore: modify behaviour to not allow unencrypted strings as decyption input

* fix add patch for enabling default v11 encryption on Linux

* chore: remove file after each test

* chore: fix patch

* chore: remove chromium patch

* chore: add linux specific tests

* chore: fix path

* chore: add checker for linuux file deletion

* chore: add dcheck back

* chore: remove reference to headless mode

* chore: remove tests for linux

* chore: edit commit message

* chore: refactor safeStorage to not be a class

* chore: remove static variable from header

* chore:  spec file remove settimeout

Co-authored-by: VerteDinde <keeleymhammond@gmail.com>
@VerteDinde VerteDinde force-pushed the trop/15-x-y-bp-feat-add-electron-safestorage-encryption-api-1628201587655 branch from 36fb76f to 302ff6a Compare August 12, 2021 02:14
@zcbenz zcbenz requested a review from VerteDinde August 19, 2021 00:02
@VerteDinde VerteDinde merged commit 24b3ee8 into 15-x-y Aug 23, 2021
@VerteDinde VerteDinde deleted the trop/15-x-y-bp-feat-add-electron-safestorage-encryption-api-1628201587655 branch August 23, 2021 22:21
@release-clerk
Copy link

release-clerk bot commented Aug 23, 2021

Release Notes Persisted

add safeStorage string encryption API

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

15-x-y backport/approved ✅ backport This is a backport PR semver/minor backwards-compatible functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants