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

Bitwarden otp #1432

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Bitwarden otp #1432

wants to merge 2 commits into from

Conversation

mandreko
Copy link

I use Bitwarden instead of 1Password, and wanted to get it working with FFXIVQuickLauncher. Bitwarden's CLI unfortunately does not have integration with the Desktop install, like 1Password does so you either have to store the password somewhere, or enter it each time the script is executed.

These scripts were based on the 1Password implementation, allowing both a manual or automatic login. Optionally, the user can store their Bitwarden master password in an encrypted blob on disk. This encrypted blob can only be decrypted on the system it was originally encrypted on (using PowerShell SecureString & DPAPI). This helps to mitigate if the user's system were compromised, an attacker could not read a plain-text master password, compromising the entire vault. However, there are still security implications to consider with this setup. It's much better than a plain-text password, or a plain-text OTP seed though.

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.

1 participant