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

real session security #17

Open
andykais opened this issue Oct 21, 2020 · 0 comments
Open

real session security #17

andykais opened this issue Oct 21, 2020 · 0 comments

Comments

@andykais
Copy link
Owner

leaving this as an open issue for jotting down thoughts. Currently sessions are reusable by storing a txt file in /tmp/bitwarden-session.txt. This is obviously not ideal, since it opens up users to an attach which looks for that open session file. I haven't figured out how to store a key somewhere on the system in such a way that I can access it from different terminals/processes and not force the user to provide a key.

Perhaps I can tie the session key to the keyring using the user's access as the permission. Its probably safe to assume that the user executing the command is safe to access that session key.

Another open is setting up a daemon. It would be easy to just assume the session key being stored in memory is "good enough" for now. This would also help with race conditions on closing the session/clearing the clipboard. A daemon is more annoying to wield though, it requires adding something to your startup scripts

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

No branches or pull requests

1 participant