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

Open specific entry from command-line parameter #32

Closed
rafi opened this issue Apr 24, 2015 · 11 comments
Closed

Open specific entry from command-line parameter #32

rafi opened this issue Apr 24, 2015 · 11 comments

Comments

@rafi
Copy link

@rafi rafi commented Apr 24, 2015

Would it be possible to immediately open QtPass with a specific entry selected and opened?
e.g.

QtPass sites/foobar.com

Thanks

@annejan annejan self-assigned this Apr 30, 2015
@annejan
Copy link
Member

@annejan annejan commented May 4, 2015

Currently this only works with the interprocess system.
When qtpass is already running you can send it such commands from the commandline.
It will even do fuzzy searching that way.

Will try to add this functionality on the first run too and let you know when this is added.
I had some problems with the lazy loading of the password store, which is why it's not include yet.

Loading

@rafi
Copy link
Author

@rafi rafi commented May 4, 2015

Thanks for the update and clarification.

Loading

@annejan
Copy link
Member

@annejan annejan commented Jun 16, 2015

This "feature" is currently giving me issues in two fields.

A) The lazy loading way the treeview works is less than ideal for "quick" searching and needs some recursive love.

B) Some OS' give weird commandline paramaters when executing from their "menu"
psn

Loading

@rafi
Copy link
Author

@rafi rafi commented Jun 16, 2015

@annejan Thanks for the update!
I'm experiencing (A) as-well, and I'd like to offer:

  1. Allow searching via path, e.g. QtPass "sites/foobar.com"
  2. Immediately display the first match on the right-pane.

Thank you so much!

Loading

@annejan
Copy link
Member

@annejan annejan commented Jul 19, 2015

Related to #26 need to revise this whole thing.

Loading

@rafi
Copy link
Author

@rafi rafi commented Aug 7, 2015

Can this get fixed now?
May I suggest an argument setting for this? i.e. QtPass -find "sites/foobar.com"

Loading

@annejan
Copy link
Member

@annejan annejan commented Aug 7, 2015

I'll look into it this weekend . .
Since I now officially on holiday I have some more time to investigate why this didn't work in the past ☀️

Currently inter-process communication is not working at-all on my MacBook .. but it is working on my Arch linux install..
But this is a nice thing to spent some time playing with tomorrow afternoon in the shade . .

Loading

@rafi
Copy link
Author

@rafi rafi commented Aug 8, 2015

Have fun! :)

Loading

@bateast
Copy link

@bateast bateast commented Aug 28, 2015

Hi,
rather than the QtPass -find "sites/foobar.com" proposition,
I would strongly suggest to use the original pass command line arguments. For searching, that would actually be QtPass find pass-names..., or QtPass show entry for a single entry match …
See http://git.zx2c4.com/password-store/about/#Commands

That would actually be great, especially as there is no command line implementation of pass for Windows …

Thanks

Loading

@jounathaen
Copy link
Member

@jounathaen jounathaen commented Oct 2, 2018

I have to disagree with the suggestion.
I simply don't see what the advantage of using qtpass would be in contrast to using pass.

pass not being available on Windows is unfortunate, but I think the aim of this project is providing a GUI, not replicating the command line interface of pass for windows.

Besides this, #88 was declined with a "wontfix".

Loading

@annejan
Copy link
Member

@annejan annejan commented Oct 4, 2018

In retrospect I think I might even see merit in #88
But the interface should be clean and well thought out . .

This is not something you want to rush into it half baked as it opens up a lot of potential issues.

Loading

@annejan annejan closed this Aug 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants