Skip to content

feat: encrypt tokens using safe storage api#1800

Merged
setchy merged 4 commits intomainfrom
feat/encrypt-tokens-safe-storage
Feb 1, 2025
Merged

feat: encrypt tokens using safe storage api#1800
setchy merged 4 commits intomainfrom
feat/encrypt-tokens-safe-storage

Conversation

@setchy
Copy link
Copy Markdown
Member

@setchy setchy commented Feb 1, 2025

Use Electron's Safe Storage API to encrypt and decrypt tokens.

Still supports backwards compatibility.

Signed-off-by: Adam Setch <adam.setch@outlook.com>
Signed-off-by: Adam Setch <adam.setch@outlook.com>
Signed-off-by: Adam Setch <adam.setch@outlook.com>
@setchy setchy changed the title Feat/encrypt tokens safe storage feat: encrypt tokens using safe storage api Feb 1, 2025
@setchy setchy added this to the Release 6.0.0 milestone Feb 1, 2025
@github-actions github-actions Bot added the enhancement New feature or enhancement to existing functionality label Feb 1, 2025
@setchy
Copy link
Copy Markdown
Member Author

setchy commented Feb 1, 2025

I ported ~95% of this from https://github.com/setchy/atlassify

The 5% newness is handling both unecrypted tokens and encrypted tokens (Atlassify launched with encryption from its first GA release)

Signed-off-by: Adam Setch <adam.setch@outlook.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 1, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
4.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@setchy setchy merged commit 2e515d0 into main Feb 1, 2025
@setchy setchy deleted the feat/encrypt-tokens-safe-storage branch February 1, 2025 13:26
Copy link
Copy Markdown
Collaborator

@bmulholland bmulholland left a comment

Choose a reason for hiding this comment

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

Nice touch, thanks for considering security

@TrayserCassa TrayserCassa mentioned this pull request Apr 29, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or enhancement to existing functionality

Development

Successfully merging this pull request may close these issues.

2 participants