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: derive PASSPHRASE for Core from the wallet #1497

Merged
merged 2 commits into from
Apr 29, 2024
Merged

Conversation

bajtos
Copy link
Member

@bajtos bajtos commented Apr 26, 2024

Create a PASSPHRASE for Station Core to let it encrypt the station identity stored on the filesystem.

The passphrase is created by signing a constant message using the private wallet key that's unique to each Station Desktop instance. This way the passphrase is unique, cannot be guessed without the knowledge of the wallet private key, and stays the same accross Station Desktop restarts.

/cc @PatrickNercessian

Links:

Create a PASSPHRASE for Station Core to let it encrypt the station
identity stored on the filesystem.

The passphrase is created by signing a constant message using the
private wallet key that's unique to each Station Desktop instance.
This way the passphrase is unique, cannot be guessed without the
knowledge of the wallet private key, and stays the same accross
Station Desktop restarts.

Signed-off-by: Miroslav Bajtoš <oss@bajtos.net>
@bajtos
Copy link
Member Author

bajtos commented Apr 26, 2024

Discussion point: should I write a test for this change?

Copy link
Member

@juliangruber juliangruber left a comment

Choose a reason for hiding this comment

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

Lgtm! To me a test isn't necessary

main/core.js Show resolved Hide resolved
@bajtos bajtos enabled auto-merge (squash) April 29, 2024 10:37
@bajtos bajtos merged commit 22600ad into main Apr 29, 2024
11 checks passed
@bajtos bajtos deleted the feat-passphrase-for-core branch April 29, 2024 10:53
@bajtos bajtos mentioned this pull request Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants