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

Unable export password store after login by device #4875

Closed
Aelliari opened this issue Aug 20, 2024 · 7 comments · Fixed by #4909
Closed

Unable export password store after login by device #4875

Aelliari opened this issue Aug 20, 2024 · 7 comments · Fixed by #4909
Labels
bug Something isn't working future Vault Issue specific to Vault that wasn't released yet Third party Needs an update from the third party library

Comments

@Aelliari
Copy link

Subject of the issue

Unable export password store after login by device

Deployment environment

  • vaultwarden version: 1.32
  • Install method: docker

  • Clients used: web vault 2024.6.2, iOS 2024.7.1 (8086)

  • Reverse proxy and version: cloudflare

  • MySQL/MariaDB or PostgreSQL version: 1:11.5.2+maria~ubu2404

  • Other relevant details: I tested only with 2fa enabled

Steps to reproduce

  1. In web vault enter email your account and press "Log in with device"
  2. After succsesful login press "tool -> Export vault -> select any format"
  3. After entering master password -> error by incorrect master password. But same time you can change setting protected master password, for example - 2fa setting, and no issue with enter master password

Expected behaviour

Succesful export password store

Actual behaviour

Master password error in export menu, but not affected an setting protected by master password

Troubleshooting data

@BlackDex
Copy link
Collaborator

BlackDex commented Aug 20, 2024

Please provide logs which are generated during these attempts.

@BlackDex BlackDex added question Further information is requested, or question from user troubleshooting There might be bug or it could be user error, more info needed labels Aug 20, 2024
@BlackDex
Copy link
Collaborator

@stefan0xC, it looks like the commit you reverted to get the masterpassword reset function working is causing issues here hehe.

@stefan0xC
Copy link
Contributor

I should really test Vaultwarden more with using a device to login... 🫣

@BlackDex
Copy link
Collaborator

Same here, but current its broken anyway using the new Bitwarden Android Beta version haha

@BlackDex
Copy link
Collaborator

Also @Aelliari, if you login using your master-password you should be able to export it just fine.

@Aelliari
Copy link
Author

Also @Aelliari, if you login using your master-password you should be able to export it just fine.

Sorry for delay, as I understand logs are no longer needed as the issue is reproducible? I'll only get to pc in a few days anyway

And yep, sorry for not mentioning that with normal login - this function works, I checked that too. Thanks for trying to help :)

@BlackDex BlackDex added bug Something isn't working future Vault Issue specific to Vault that wasn't released yet Third party Needs an update from the third party library and removed question Further information is requested, or question from user troubleshooting There might be bug or it could be user error, more info needed labels Aug 26, 2024
@stefan0xC
Copy link
Contributor

I should really test Vaultwarden more with using a device to login... 🫣

to replicate the issue it's also important to not use the master password to unlock the android device but just a pin or biometrics. (so even if i did use a login with device I probably would have missed this).

BlackDex added a commit to BlackDex/vaultwarden that referenced this issue Aug 31, 2024
- Updated the web-vault to fix an issue with personal export.
  Thanks to @stefan0xC for patching this.
  Fixes dani-garcia#4875
- Updated crates to there latest version
- Updated the GitHub Actions
- Updated the xx image to the latest version

Signed-off-by: BlackDex <black.dex@gmail.com>
BlackDex added a commit to BlackDex/vaultwarden that referenced this issue Aug 31, 2024
- Updated the web-vault to fix an issue with personal export.
  Thanks to @stefan0xC for patching this.
  Fixes dani-garcia#4875
- Updated crates to there latest version
- Updated the GitHub Actions
- Updated the xx image to the latest version

Signed-off-by: BlackDex <black.dex@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working future Vault Issue specific to Vault that wasn't released yet Third party Needs an update from the third party library
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants