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

Make password storage optional #1512

Closed
outfrost opened this issue Jun 5, 2019 · 3 comments
Closed

Make password storage optional #1512

outfrost opened this issue Jun 5, 2019 · 3 comments
Assignees
Labels
migrated Migrated to Discourse product-direction

Comments

@outfrost
Copy link

outfrost commented Jun 5, 2019

Continuing the problem and discussion from #681, it seems that development in Mailspring's dependencies regarding that issue has mostly stalled. I reckon something can still be done here, though.

Could you make it an option to not store passwords on disk at all?

It seems that node-keytar relies on libsecret on Linux, and libsecret in turn relies on gnome-keyring, as the only full/viable existing implementation of the org.freedesktop.secrets DBus functionality.

For me, as a KDE Plasma user, installing gnome-keyring, or using any of the few workarounds that I've found for this problem, is undesirable. I would simply prefer Mailspring to not store my password offline, and to only hold it in RAM while I'm logged in, for the purpose of continuous sync. FWIW, it is an option in Thunderbird.

Originally posted by @outfrost in #681 (comment)

@bengotow
Copy link
Collaborator

Hey! Hmm, technically this wouldn't be too hard really... the app essentially just reads your passwords from the keychain and hands them off to the sync workers that open the IMAP connections. I'll take a closer look and see. I think the trickiest part is that it essentially means inserting a delay where the sync process hasn't been started / given a password yet, and the app might be a little unstable in that state. Will take a look...

@bengotow bengotow self-assigned this Jun 17, 2019
@CodeMouse92 CodeMouse92 added the audit Auditing issue label Jan 12, 2021
@foundry376-bot
Copy link

This issue has been mentioned on Mailspring Community. There might be relevant details there:

https://community.getmailspring.com/t/make-password-storage-optional/675/1

@CodeMouse92 CodeMouse92 added migrated Migrated to Discourse and removed audit Auditing issue labels Feb 17, 2021
@CodeMouse92
Copy link
Contributor

We are in the process of migrating issues to Discourse, which can better facilitate discussion and discovery, and so GitHub Issues can focus on issues that are confirmed and slated for resolution in the near term. Learn more about the changes here.

As part of this, we've migrated this issue to Discourse:

https://community.getmailspring.com/t/make-password-storage-optional/675

Please consider joining that community and continuing the discussion there! Votes on the feature suggestion on Discourse will increase the likelihood we implement this.

@outfrost: if you join and reply to the issue, the moderators can make an effort to reassign your posts to you, so you get the credit for them.

We're closing and locking the issue here as part of this migration. Rest assured, this doesn't mean the issue is being discarded or ignored.

We hope to see you on Discourse soon!

-The Mailspring Team

@Foundry376 Foundry376 locked and limited conversation to collaborators Feb 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
migrated Migrated to Discourse product-direction
Projects
None yet
Development

No branches or pull requests

4 participants