Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add electron.safeStorage encryption API #30430

Conversation

trop[bot]
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 in the last 24 hours 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 in the last 24 hours 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.

None yet

5 participants