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

Allow vault search to include ports in URI matching #13134

Open
1 task done
calebvisage opened this issue Jan 29, 2025 · 1 comment · May be fixed by #13136
Open
1 task done

Allow vault search to include ports in URI matching #13134

calebvisage opened this issue Jan 29, 2025 · 1 comment · May be fixed by #13136
Labels
bug desktop Desktop Application

Comments

@calebvisage
Copy link

Steps To Reproduce

  1. Create a new dummy login item with a URI like "192.168.1.2:8000" (include a port).
  2. Try to search for ":8000" (the dummy item just created)
  3. Bitwarden cannot find it.

Expected Result

It should find the item.

Actual Result

It does not find the item, because uriExtractor strips the port out of the uri.

Screenshots or Videos

Image

Additional Context

can create PR

Operating System

Windows, macOS, Linux

Operating System Version

No response

Installation method

Direct Download (from bitwarden.com)

Build Version

2025.1.0

Issue Tracking Info

  • I understand that work is tracked outside of GitHub. A PR will be linked to this issue should one be opened to address it, but Bitwarden doesn't use fields like "assigned", "milestone", or "project" to track progress.
@calebvisage calebvisage added bug desktop Desktop Application labels Jan 29, 2025
@bitwarden-bot
Copy link

Thank you for reporting this issue! We've added this to our internal tracking system.
ID: PM-17723

@calebvisage calebvisage changed the title Cannot search for port in vault login item URI Allow vault search to include ports in URI matching Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug desktop Desktop Application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants