-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
UI can't handle passwords with periods in their name #520
Comments
I have also encountered this but while adding website passords. Encountered in Windows 10, QTPass version 1.3.2, manually configured gpg "C:/Program Files (x86)/GnuPG/bin/gpg.exe" git "C:/Program Files/Git/bin/git.exe" pwgen "C:/Program Files (x86)/PWGen/PWGen.exe", native git/gpg option used, password store path "C:/Users/USERNAME/password-store/", "Use git" checked, "Automatically add .gpg-id" checked", "Automatically push/pull" both unchecked. |
On GNU/Linux (using Pass, but not using Git):
So, on GNU/Linux the issue of the renaming UI is the same as described above for Windows, but the files get renamed correctly, so the entries do not disappear in the list UI. |
... and
Can you reproduce this? I can't and the relevant code in realpass.cpp:118 looks fine to me. |
I wasn't using real pass, I was imitating it. I suspect it has something to do with Qt's baseName() function. You should probably use completeBaseName() instead. |
Ah, yes, in imitatepass.cpp:347 it's the same problem I fixed in #558 . The function is a bit messy though; I think it would be better if the author (or anyone familiar with the code) could have a look at it. |
I had another look and fixed the "baseName" problem in #559 . I still cannot see how the old suffix ".account" could be kept if the new name "different-name" is correctly used. Before the fixes it should have either become "my.gpg" or "different-name.gpg". Or "different-name" already contained ".account"...? @woodenphone 's bug could be related to real pass (realpass.cpp:60). Adding the suffix is left to pass, there. I'll stop digging in code I'm not familiar with :). |
@ichthyosaurus About the suffix bug, I was using v1.3.2 (latest release version) and I believe this was the cause of the error: Line 1010 in d242041
Replacing baseName() with the inserted name, and then for some reason removing the .gpg extension from the new file. That just looks very wrong to me. But that was fixed with a commit in June. It's just that the latest release is extremely old... And nobody (myself included) is gonna build stuff from source on Windows. So yeah, don't worry about that. |
I suppose this means I can close this issue, but man, the fact that the last release binaries are over a year old just really rubs me the wrong way. That's no way to maintain a password manager. |
Let's say, I have a password named
my.account
, so it's stored in a file namedmy.account.gpg
. I want to rename it todifferent-name
, so I click on "Rename password".different-name.gpg
, it saves it asdifferent-name.account
! Then it no longer shows up on the password list.Tested on Windows 10, without Git or native Pass.
The text was updated successfully, but these errors were encountered: