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

Browser integration: multiple URLs #1821

Closed
Thulium-Drake opened this issue Apr 6, 2018 · 19 comments
Closed

Browser integration: multiple URLs #1821

Thulium-Drake opened this issue Apr 6, 2018 · 19 comments

Comments

@Thulium-Drake
Copy link

Thulium-Drake commented Apr 6, 2018

Expected Behavior

Allow for multiple specifications for URLs on which the use of a credential is allowed.

Current Behavior

You can only set 1 url (or wildcard) on a credential, even if the systems on those urls share a common authentication source (e.g. LDAP)

Possible Solution

I think the best implementation for this would be the same as how one would 'describe' Auto-Type sequences and windows, like so:

URL: https://auth.example.com
URL-1: https://auth.examplesotherdomain.com
URL-2: https://somehost (which would be in the local network's search domain)

(P.S. for those who might not know, https://*.example.com is a valid URL ;-), but that doesn't allow for mulitple domains or the single word DNS names)

Context

I'm trying to migrate from all auto-type to a more integrated solution and this is stopping me from doing so :-)

Debug Info

KeePassXC - Version 2.3.1
Revision: 2fcaeea

Libraries:

  • Qt 5.7.1
  • libgcrypt 1.7.6-beta

Operating system: Debian GNU/Linux 9 (stretch)
CPU architecture: x86_64
Kernel: linux 4.9.0-6-amd64

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • Legacy Browser Integration (KeePassHTTP)
  • SSH Agent
  • YubiKey
@varjolintu
Copy link
Member

See this #1769. When it's merged I will do any necessary changes to the browser integration side.

@Thulium-Drake
Copy link
Author

Thanks for the prompt reply! That does indeed seem to accomodate this usecase quite well!

@droidmonkey
Copy link
Member

droidmonkey commented Apr 8, 2018

Edit: I see now that this is targeted for the browser plugin side.

@Thulium-Drake
Copy link
Author

Hi Guys,

Thanks for integrating this feature! From what I can see in 2.3.4 this works like a charm!

@twwd
Copy link

twwd commented Mar 25, 2019

How does it work? I can't find any details about that.

@droidmonkey
Copy link
Member

droidmonkey commented Mar 25, 2019

This is an issue, not a code contribution. (and its closed, no work was done here)

@twwd
Copy link

twwd commented Mar 25, 2019

I know that this is just an issue but @Thulium-Drake wrote:

Thanks for integrating this feature! From what I can see in 2.3.4 this works like a charm!

The method described in the linked PR #1769 doesn't work (the PR is closed anyways) and I couldn't find any information in the changelog of version 2.3.4. So I want to know how it works for @Thulium-Drake.

@Thulium-Drake
Copy link
Author

@twwd

I have something like this:

https://login.example.com|https://login.otherexample.net 

Set as the URL for an entry and the browser plugin picks up on it.

@brainplot
Copy link
Contributor

brainplot commented Aug 27, 2019

https://login.example.com|https://login.otherexample.net 

@Thulium-Drake Is that really how it's supposed to work or are you basically injecting regex syntax?

@Thulium-Drake
Copy link
Author

This is how I'm using it at the moment, but I'm not sure what it will do underwater :-)

@AlexanderAmelkin
Copy link

I have something like this:
https://login.example.com|https://login.otherexample.net
Set as the URL for an entry and the browser plugin picks up on it.

This definitely does NOT work for me with KeepassXC 2.5.1 and KeepassXC-Browser 1.5.3.
The extension just can't find the record at all.

@varjolintu
Copy link
Member

varjolintu commented Nov 19, 2019

@AlexanderAmelkin That worked becase of an earlier bug. Anyone, do not use it.

@YogSottot
Copy link

YogSottot commented Nov 19, 2019

This definitely does NOT work for me with KeepassXC 2.5.1

@AlexanderAmelkin You should use this #3558

@AlexanderAmelkin
Copy link

@YogSottot, thank you so much! It doesn't fully work for me as expected (described there), but it solves the problem.

@fourstepper
Copy link

Is there a way to use regex with the URLs?

@varjolintu
Copy link
Member

@fourstepper At this point, no.

@fourstepper
Copy link

thanks for the swift reply - that's not a big deal

@JohnLGalt
Copy link

JohnLGalt commented Jul 15, 2021

You want a comment from over 2 years ago to be edited now, after the feature has been properly implemented into KPXC?

@Jorricks
Copy link

Alright alright. Maybe that is expecting a bit to much 😅 it's just that I was trying that method and couldn't get it to work. Guess I should have scrolled down.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

10 participants