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.

Sign up for GitHub

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

Jump to bottom

keepassxc-cli show should show all matching entries #1967

Open
amtlib-dot-dll opened this issue May 16, 2018 · 1 comment
Open

keepassxc-cli show should show all matching entries #1967

amtlib-dot-dll opened this issue May 16, 2018 · 1 comment
Assignees
@louib
Labels
feature: CLI new feature

Comments

@amtlib-dot-dll
Copy link

amtlib-dot-dll commented May 16, 2018 •
edited by droidmonkey

Expected Behavior

$ keepassxc-cli ls Test.kdbx <<<'Password'
Insert password to unlock AnotherDB.kdbx: 
Test
Test
$ keepassxc-cli show Test <<<'Password'
Insert password to unlock AnotherDB.kdbx: 
Title: Test
UserName: test1@contoso.com
Password: p@ssW0rd1
URL: https://www1.contoso.com/login
Notes: There is another one with the same title!
Title: Test
UserName: tes2@contoso.com
Password: p@ssW0rd2
URL: https://www2.contoso.com/login
Notes: This is the other one

Current Behavior

$ keepassxc-cli ls Test.kdbx <<<'Password'
Insert password to unlock AnotherDB.kdbx: 
Test
Test
$ keepassxc-cli show Test <<<'Password'
Insert password to unlock AnotherDB.kdbx: 
Title: Test
UserName: test1@contoso.com
Password: p@ssW0rd1
URL: https://www1.contoso.com/login
Notes: There is another one with the same title!

Possible Solution

Modify the code but I don't know any technical background 😂

Steps to Reproduce (for bugs)

  1. Create a database with entries described above
  2. Run keepassxc-cli show as above
@TheZ3ro TheZ3ro added this to the v2.4.0 milestone May 16, 2018
coelebs added a commit to coelebs/keepassxc that referenced this issue Oct 5, 2018
@coelebs
Make keepassxc-cli show return all matching entries.
47e6b79 
As mentioned in issue keepassxreboot#1967 show should return all matching items
instead of the first one. If a UUID is passed to show it will return one
entry if there is a match. For a path or name it will return all
matching entries.

TODO: Replace us of QMap.unite() to a function that skips duplicates.
coelebs added a commit to coelebs/keepassxc that referenced this issue Oct 5, 2018
@coelebs
Make keepassxc-cli show return all matching entries.
0c83892 
As mentioned in issue keepassxreboot#1967 show should return all matching items
instead of the first one. If a UUID is passed to show it will return one
entry if there is a match. For a path or name it will return all
matching entries.

TODO: Replace us of QMap.unite() to a function that skips duplicates.
@coelebs coelebs mentioned this issue Oct 7, 2018
Closed
coelebs added a commit to coelebs/keepassxc that referenced this issue Oct 13, 2018
@coelebs
Make keepassxc-cli show return all matching entries.
ba20ba8 
As mentioned in issue keepassxreboot#1967 show should return all matching items
instead of the first one. If a UUID is passed to show it will return one
entry if there is a match. For a path or name it will return all
matching entries.

TODO: Replace us of QMap.unite() to a function that skips duplicates.
coelebs added a commit to coelebs/keepassxc that referenced this issue Oct 13, 2018
@coelebs
Make keepassxc-cli show return all matching entries.
7cd9b7b 
As mentioned in issue keepassxreboot#1967 show should return all matching items
instead of the first one. If a UUID is passed to show it will return one
entry if there is a match. For a path or name it will return all
matching entries.

TODO: Replace us of QMap.unite() to a function that skips duplicates.
@droidmonkey droidmonkey modified the milestones: v2.4.0, v2.4.1 Jan 31, 2019
@droidmonkey droidmonkey modified the milestones: v2.4.1, v2.5.0 Apr 12, 2019
@droidmonkey droidmonkey changed the title keepassxc-cli show only show the first entry when two available keepassxc-cli show should show all matching entries Apr 25, 2019
@droidmonkey
Copy link
Member

This should be implemented using the EntrySearcher class

@phoerious phoerious removed this from the v2.5.0 milestone Oct 26, 2019
@louib louib mentioned this issue Jan 4, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@louib louib

Labels
feature: CLI new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make keepassxc-cli show return all matching entries. coelebs/keepassxc
5 participants
@TheZ3ro @phoerious @droidmonkey @louib @amtlib-dot-dll