Skip to content

Windows-related fixes.#17

Merged
annejan merged 17 commits intoIJHack:masterfrom
rdoeffinger:master
Mar 29, 2015
Merged

Windows-related fixes.#17
annejan merged 17 commits intoIJHack:masterfrom
rdoeffinger:master

Conversation

@rdoeffinger
Copy link
Contributor

@rdoeffinger rdoeffinger commented Mar 29, 2015

The patches I sent as issues before as pull request.

@rdoeffinger
Windows: Make static builds truly static, enable NX and ASLR.
5b43c6f 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Enable UI elements when the process e.g. fails to start.
09d1567 
There will be no finish() event in that case.
This can happen if the program to start does not
exist or has wrong permissions.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Avoid dependency in "sh", let Qt handle argument splitting.
1cf25b4 
This makes the program work on Windows.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Add --use-agent to allow secret keys with password or smartcard with …
b910c6c 
…PIN.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Implement edit, insert and delete.
7b047f7 
A bit quick-and-dirty, but it seems to work.
Note: The usePass variant is not properly tested.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Copy link
Contributor Author

rdoeffinger commented Mar 29, 2015

Sorry, was too lazy to create branches. This pull request now also contains edit/insert/delete feature implementation.
If you prefer I can make a branch with only the Windows fixes.

@rdoeffinger
Add quotes around executable name in QProcess::start.
f993eac 
This fixes failures if the name contains spaces.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Normalize path to password store to end in directory separator.
2ea9813 
Makes it easier and more reliable to use.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Support ini file in application directory.
b896613 
This is useful for use from a USB stick.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Add support for gpgHome setting.
64c01e8 
It is used to set the GNUPGHOME environment variable.
This allows to specify the GnuPG configuration and
keyring location, especially for use on USB stick.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Simplify, fix editing if passStore is relative path.
7a20668 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Fix password files containing spaces.
6addb97 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Use multiline input dialog if available.
bbbdbe4 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Ensure error output is not overwritten on edit/add.
4c0727c 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Use QScopedPointer to reduce memleaks.
8c21e7e 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Fix Windows path-search special-case.
e26c3d1 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Reduce variable scope.
05b62f0 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@rdoeffinger
Remove unused class variable.
1fe14ea 
Signed-off-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de>
@annejan
Copy link
Member

annejan commented Mar 29, 2015

Adding now, will replace popups with multiline edit (for password) and merge ..

annejan added a commit that referenced this pull request Mar 29, 2015
@annejan
Merge pull request #17 from rdoeffinger/master
bdf482a 
Windows-related fixes.
Basic editing and adding.
@annejan annejan merged commit bdf482a into IJHack:master Mar 29, 2015
@rdoeffinger
Copy link
Contributor Author

rdoeffinger commented Mar 30, 2015

Thanks!
Just to make sure you didn't miss it: Since bbbdbe4 the password input dialog is multi-line if you use Qt 5.2 or later.
So there should be no need to replace it unless you want to fully support earlier versions (I kept the single line edit as fallback since I wasn't sure dropping support for such recent versions was reasonable).

@annejan
Copy link
Member

annejan commented Mar 30, 2015

Ah yes, noticed it later on my main laptop (with up-to-date Qt).
In the develop branch I have made some changes last month which I would like to intergrate that used the multi-line field used for viewing, for editing too.
So I might still change the editing to a smoother (no popup) experience ;)
Thanks for your work, this has gotten me back into working on this project some more too :)

@rdoeffinger
Copy link
Contributor Author

rdoeffinger commented Mar 30, 2015

Just for reference, I had (besides lazyness of course ;) ) a few half-good reasons not to use the existing field.
For example I felt that it is fairly overloaded already, since both errors, error messages and decrypted output end up in there. This seems particularly bad when the editing/adding/encrypting failed and you need to display the error in the same thing where you just entered the text...
Then specifically for editing, there is a bit of complex interaction with the clipboard features that remove e.g. the password. Would you when editing ignore that setting and display the full text?
Also some people might want an option to never show passwords, with the dialog it is fairly simple to just pop it up twice and ask to enter the same thing.
I'm sure it's quite possible to come up with something better than I implemented, it just felt like it might have to come with a bigger redesign of the interface.
Btw: one small thing I noticed: "pass" seems to add an extra newline at the end of the passwords, and it clearly assumes it to exist when it prints the value. I don't really like that behaviour, but it might make sense to align qtpass with that anyway....

@annejan annejan self-assigned this Mar 30, 2015
annejan added a commit that referenced this pull request Jun 1, 2016
@annejan
Merge pull request #17 from rdoeffinger/master
bde1746 
Windows-related fixes.
Basic editing and adding.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@annejan annejan

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rdoeffinger @annejan